Skip to content

chore: relax Go version constraint to 1.23#4964

Merged
zrbecker merged 2 commits intocometbft:mainfrom
zrbecker:zrb/relax_go_version
Feb 20, 2025
Merged

chore: relax Go version constraint to 1.23#4964
zrbecker merged 2 commits intocometbft:mainfrom
zrbecker:zrb/relax_go_version

Conversation

@zrbecker
Copy link
Copy Markdown

@zrbecker zrbecker commented Feb 19, 2025

This PR relaxes to Go version from 1.23 from 1.23.5. This should implicitly use the latest patch version by default, but let dependent modules control of the patch model. Although it's recommended that the latest patch is used in general.

This avoids the need to constantly update this value on every Go patch upgrade.

Similar change done in cometbft-db here (cometbft/cometbft-db#245)


PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments

@zrbecker zrbecker marked this pull request as ready for review February 19, 2025 17:23
@zrbecker zrbecker requested a review from a team as a code owner February 19, 2025 17:23
@zrbecker zrbecker requested a review from a team February 19, 2025 17:23
@zrbecker zrbecker requested a review from a team as a code owner February 19, 2025 17:23
@zrbecker zrbecker added this pull request to the merge queue Feb 20, 2025
Merged via the queue into cometbft:main with commit a8b48c7 Feb 20, 2025
31 checks passed
@zrbecker zrbecker deleted the zrb/relax_go_version branch February 20, 2025 15:47
@zrbecker zrbecker added backport-to-v0.38.x Tell Mergify to backport the PR to v0.38.x backport-to-v1.x labels Feb 20, 2025
mergify bot pushed a commit that referenced this pull request Feb 20, 2025
This PR relaxes to Go version from 1.23 from 1.23.5. This should
implicitly use the latest patch version by default, but let dependent
modules control of the patch model. Although it's recommended that the
latest patch is used in general.

This avoids the need to constantly update this value on every Go patch
upgrade.

Similar change done in cometbft-db here
(cometbft/cometbft-db#245)

---

#### PR checklist

- [x] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [x] Updated relevant documentation (`docs/` or `spec/`) and code
comments

(cherry picked from commit a8b48c7)

# Conflicts:
#	test/e2e/docker/Dockerfile
#	test/e2e/docker/Dockerfile.debug
mergify bot pushed a commit that referenced this pull request Feb 20, 2025
This PR relaxes to Go version from 1.23 from 1.23.5. This should
implicitly use the latest patch version by default, but let dependent
modules control of the patch model. Although it's recommended that the
latest patch is used in general.

This avoids the need to constantly update this value on every Go patch
upgrade.

Similar change done in cometbft-db here
(cometbft/cometbft-db#245)

---

#### PR checklist

- [x] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [x] Updated relevant documentation (`docs/` or `spec/`) and code
comments

(cherry picked from commit a8b48c7)

# Conflicts:
#	.github/workflows/go-version.env
#	DOCKER/Dockerfile
#	README.md
#	api/go.mod
#	docs/guides/go-built-in.md
#	docs/guides/go.md
#	go.mod
#	go.work
#	test/e2e/docker/Dockerfile
#	test/e2e/docker/Dockerfile.debug
zrbecker pushed a commit that referenced this pull request Feb 24, 2025
This PR relaxes to Go version from 1.23 from 1.23.5. This should
implicitly use the latest patch version by default, but let dependent
modules control of the patch model. Although it's recommended that the
latest patch is used in general.

This avoids the need to constantly update this value on every Go patch
upgrade.

Similar change done in cometbft-db here
(cometbft/cometbft-db#245)

---

#### PR checklist

- [x] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [x] Updated relevant documentation (`docs/` or `spec/`) and code
comments
<hr>This is an automatic backport of pull request #4964 done by
[Mergify](https://mergify.com).

---------

Co-authored-by: Zachary Becker <zachary@interchainlabs.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-to-v0.38.x Tell Mergify to backport the PR to v0.38.x

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants