Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update golang.org/x/exp, fixing slice sorting to match standard library #210

Merged
merged 1 commit into from
Aug 9, 2023

Conversation

cce
Copy link
Contributor

@cce cce commented Aug 2, 2023

This updates the dependency on the slices package from golang.org/x/exp used here to match the upcoming Go 1.21 standard library API, which uses a different comparison function.

golang/exp@302865e
golang/go#61374

@cce cce closed this Aug 2, 2023
@cce
Copy link
Contributor Author

cce commented Aug 2, 2023

Thinking about it more, this should probably just be closed unless a critical mass of projects want to use the new exp/slices package, or until everyone adopts Go 1.21 standard slices, making this PR moot

@marten-seemann
Copy link
Contributor

@cce Why are you closing this PR?

@cce
Copy link
Contributor Author

cce commented Aug 3, 2023

oh OK! I'll re-open it — I started to think it might be more trouble than it was worth

@cce cce reopened this Aug 3, 2023
@marten-seemann marten-seemann changed the title Update golang.org/x/exp slice comparison to match standard library update golang.org/x/exp, fixing slice sorting to match standard library Aug 3, 2023
@marten-seemann
Copy link
Contributor

Thank you @cce! This will be included in the go-libp2p v0.30 release.

@paralin
Copy link
Contributor

paralin commented Aug 7, 2023

Dupe of #207

@marten-seemann marten-seemann merged commit c6da2e6 into multiformats:master Aug 9, 2023
11 checks passed
@cce cce deleted the use-std-exp-sortfunc branch August 25, 2023 16:52
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.

3 participants