Skip to content

refactor!: upgrade golang.org/x/exp to latest#83

Merged
julienrbrt merged 2 commits into
cosmos:mainfrom
prasincs:upgrade-exp-slices
Aug 17, 2023
Merged

refactor!: upgrade golang.org/x/exp to latest#83
julienrbrt merged 2 commits into
cosmos:mainfrom
prasincs:upgrade-exp-slices

Conversation

@prasincs

@prasincs prasincs commented Aug 7, 2023

Copy link
Copy Markdown
Contributor

Recently golang.org/x/exp package decided to change function signature for SortFunc to align better with cmp package
reasoning is here:
golang/go#60091 (comment)

this is likely going to be standard for Go 1.21 and going forward

@prasincs prasincs requested a review from a team August 7, 2023 23:48
@prasincs prasincs changed the title Upgrade golang.org/x/exp to latest chore: Upgrade golang.org/x/exp to latest Aug 7, 2023

@tac0turtle tac0turtle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you

@julienrbrt

Copy link
Copy Markdown
Contributor

Given that people when depending on cosmos/gogoproto and trying to bump to latest pseudo version of golang.org/x/exp will currently get an error. I think it is worth a changelog.

@julienrbrt julienrbrt changed the title chore: Upgrade golang.org/x/exp to latest refactor!: upgrade golang.org/x/exp to latest Aug 17, 2023
@julienrbrt julienrbrt merged commit 19fa753 into cosmos:main Aug 17, 2023
beer-1 added a commit to initia-labs/cosmos-sdk that referenced this pull request Dec 8, 2023
beer-1 added a commit to initia-labs/cosmos-sdk that referenced this pull request Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants