Skip to content

chore: replace go-hex with encoding/hex - #1410

Draft
oozan wants to merge 1 commit into
uptrace:masterfrom
oozan:chore-replace-go-hex-1784799027
Draft

chore: replace go-hex with encoding/hex#1410
oozan wants to merge 1 commit into
uptrace:masterfrom
oozan:chore-replace-go-hex-1784799027

Conversation

@oozan

@oozan oozan commented Jul 23, 2026

Copy link
Copy Markdown

What changed

  • Replaced the unmaintained github.com/tmthrgd/go-hex dependency with Go's standard encoding/hex package
  • Removed go-hex from the root module dependencies
  • Added regression coverage for empty, single, and incremental writes

The two packages expose compatible EncodedLen and Encode functions, so SQL byte encoding behavior remains unchanged while removing an abandoned supply-chain dependency.

Testing

  • go test ./internal ./schema
  • go vet ./internal ./schema
  • git diff --check

Fixes #1385

AI assistance was used to prepare this focused change. I reviewed the implementation and test coverage and am prepared to address review feedback.

@github-actions

Copy link
Copy Markdown

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. If there is no update within the next 7 days, this pr will be closed. Please feel free to give a status update now, ping for review, when it's ready. Thank you for your contributions!

@github-actions github-actions Bot added the stale label Aug 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace unmaintained github.com/tmthrgd/go-hex with stdlib encoding/hex

1 participant