Skip to content

build(deps): bump to go 1.24 - #4509

Merged
julienrbrt merged 33 commits into
mainfrom
julien/124
Mar 8, 2025
Merged

build(deps): bump to go 1.24#4509
julienrbrt merged 33 commits into
mainfrom
julien/124

Conversation

@julienrbrt

@julienrbrt julienrbrt commented Feb 14, 2025

Copy link
Copy Markdown
Member

Bump cli + app to Go 1.24 and use the new tool pattern: https://go.dev/doc/modules/managing-dependencies#tools

@github-actions github-actions Bot added component:ci CI/CD workflow and automated jobs. component:configs type:internal labels Feb 14, 2025
@julienrbrt julienrbrt added the skip-changelog Don't check changelog for new entries label Feb 14, 2025
@julienrbrt

Copy link
Copy Markdown
Member Author

Super weird, looks like the library used by json library <cosmossdk.io/log doesn't support Go 1.24 yet.

@julienrbrt

Copy link
Copy Markdown
Member Author

We can go further after bytedance/sonic#741

julienrbrt added a commit that referenced this pull request Feb 17, 2025
julienrbrt added a commit that referenced this pull request Feb 17, 2025
* feat(services): add faucet fee config

* cl

* pin 1.23 to ci (ref #4509)

* fix
mergify Bot pushed a commit that referenced this pull request Feb 17, 2025
* feat(services): add faucet fee config

* cl

* pin 1.23 to ci (ref #4509)

* fix

(cherry picked from commit 4abea76)

# Conflicts:
#	ignite/config/chain/base/config.go
julienrbrt added a commit that referenced this pull request Feb 17, 2025
* feat(services): add faucet fee config (#4513)

* feat(services): add faucet fee config

* cl

* pin 1.23 to ci (ref #4509)

* fix

(cherry picked from commit 4abea76)

# Conflicts:
#	ignite/config/chain/base/config.go

* conflict

---------

Co-authored-by: Julien Robert <julien@rbrt.fr>
@julienrbrt
julienrbrt marked this pull request as ready for review March 7, 2025 12:07
@julienrbrt julienrbrt removed the skip-changelog Don't check changelog for new entries label Mar 7, 2025
@julienrbrt
julienrbrt marked this pull request as draft March 7, 2025 12:56
@julienrbrt
julienrbrt marked this pull request as ready for review March 7, 2025 13:44
@julienrbrt

Copy link
Copy Markdown
Member Author

@Pantani should be good to review again

@julienrbrt
julienrbrt enabled auto-merge (squash) March 7, 2025 17:10

@Pantani Pantani 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.

LGMT
ACK, tested with a new chain and a chain scaffolded with a preview Golang version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants