Skip to content

Comments

Fix encoder naming convention and restore JPEG redirect test#66

Merged
thraxil merged 3 commits intowebpfrom
copilot/sub-pr-65
Feb 11, 2026
Merged

Fix encoder naming convention and restore JPEG redirect test#66
thraxil merged 3 commits intowebpfrom
copilot/sub-pr-65

Conversation

Copy link
Contributor

Copilot AI commented Feb 11, 2026

Addresses review feedback on PR #65 regarding naming consistency and test coverage.

Changes

  • Renamed encodeWEBPencodeWebP to match existing encoder naming pattern (encodeJPEG, encodeGIF, encodePNG)
  • Updated extencoders map to reference corrected function name
  • Restored JPEG redirect test case verifying .jpeg.jpg redirect behavior (HTTP 301)
// Before
func encodeWEBP(out io.Writer, in image.Image) error {
    return nativewebp.Encode(out, in, nil)
}

// After
func encodeWebP(out io.Writer, in image.Image) error {
    return nativewebp.Encode(out, in, nil)
}

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits February 11, 2026 21:20
- Rename encodeWEBP to encodeWebP for consistency
- Update extencoders map reference
- Add back JPEG redirect test case

Co-authored-by: thraxil <7821+thraxil@users.noreply.github.com>
Co-authored-by: thraxil <7821+thraxil@users.noreply.github.com>
Copilot AI changed the title [WIP] Add support for WebP image format Fix encoder naming convention and restore JPEG redirect test Feb 11, 2026
Copilot AI requested a review from thraxil February 11, 2026 21:22
@thraxil thraxil marked this pull request as ready for review February 11, 2026 21:24
@thraxil thraxil merged commit 1863742 into webp Feb 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants