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

chore(deps): upgrade cometBFT #2608

Merged
merged 4 commits into from
Mar 18, 2025
Merged

chore(deps): upgrade cometBFT #2608

merged 4 commits into from
Mar 18, 2025

Conversation

abi87
Copy link
Collaborator

@abi87 abi87 commented Mar 17, 2025

We upgraded CometBFT dep to pull in a few security fixes. See berachain/cometbft#25 for context.
As a consequence we upgraded go version too, to 1.23.6

@abi87 abi87 requested a review from a team as a code owner March 17, 2025 13:44
@abi87 abi87 self-assigned this Mar 17, 2025
Copy link

codecov bot commented Mar 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.61%. Comparing base (8dee8a0) to head (a14d04e).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2608   +/-   ##
=======================================
  Coverage   58.61%   58.61%           
=======================================
  Files         340      340           
  Lines       15272    15272           
  Branches       20       20           
=======================================
  Hits         8951     8951           
  Misses       5652     5652           
  Partials      669      669           
🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Dockerfile Outdated
@@ -18,7 +18,7 @@
### Stage 0 - Build Arguments ###
#######################################################

ARG GO_VERSION=1.23.5
ARG GO_VERSION=6
Copy link
Contributor

Choose a reason for hiding this comment

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

uhhhh @abi87??

Copy link
Collaborator Author

@abi87 abi87 Mar 17, 2025

Choose a reason for hiding this comment

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

typo, thanks for spotting this. Fixed

go.mod Outdated
go 1.23.5
go 1.23.6

toolchain go1.23.7
Copy link
Contributor

@fridrik01 fridrik01 Mar 17, 2025

Choose a reason for hiding this comment

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

whats the reason for using a newer toolchain than go version?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Picked automagically by my machine, reverted

@@ -45,12 +45,12 @@ require (
github.com/spf13/afero v1.11.0
github.com/spf13/cast v1.7.0
github.com/spf13/cobra v1.8.1
github.com/spf13/pflag v1.0.5
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Replaces #2586

Comment on lines -52 to -53
golang.org/x/crypto v0.32.0
golang.org/x/sync v0.10.0
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Contributor

@nidhi-singh02 nidhi-singh02 left a comment

Choose a reason for hiding this comment

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

lgtm!

@abi87 abi87 merged commit eead674 into main Mar 18, 2025
19 checks passed
@abi87 abi87 deleted the update-cometBFT branch March 18, 2025 09:40
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.

3 participants