Skip to content

fix: Replace wk8/go-ordered-map with pb33f/ordered-map#16

Merged
erezrokah merged 1 commit into
cqmainfrom
fix/pb33f-ordered-map
Jul 3, 2026
Merged

fix: Replace wk8/go-ordered-map with pb33f/ordered-map#16
erezrokah merged 1 commit into
cqmainfrom
fix/pb33f-ordered-map

Conversation

@erezrokah

Copy link
Copy Markdown
Member

Migrates the ordered-map dependency from wk8/go-ordered-map/v2 to the API-compatible pb33f/ordered-map/v2, matching upstream invopop/jsonschema v0.14.0, so consumers pinning invopop v0.14.0 (e.g. anthropic-sdk-go v1.52.0) build against this fork without an OrderedMap type mismatch.

@erezrokah erezrokah marked this pull request as ready for review July 3, 2026 17:10
@erezrokah erezrokah force-pushed the fix/pb33f-ordered-map branch 3 times, most recently from 8ca148a to 738034d Compare July 3, 2026 17:32
Matches upstream invopop/jsonschema v0.14.0, which switched to the
API-compatible pb33f/ordered-map/v2 fork. Required so consumers pinning
invopop v0.14.0 (e.g. anthropic-sdk-go v1.52.0) build against this fork
without an OrderedMap type mismatch.

The go.mod directive moves to go 1.24 (pb33f requirement), so CI is
updated too: track the Go version via go-version-file, pin golangci-lint
v1.64.8 (last v1 line, go 1.24 compatible) with the existing v1 config,
and run Lint on pushes to cqmain (the default branch).
@erezrokah erezrokah force-pushed the fix/pb33f-ordered-map branch from 738034d to 4652363 Compare July 3, 2026 17:34
@erezrokah erezrokah merged commit 45e7e20 into cqmain Jul 3, 2026
7 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.

1 participant