-
Notifications
You must be signed in to change notification settings - Fork 191
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
Pin buf plugin version #2395
base: main
Are you sure you want to change the base?
Pin buf plugin version #2395
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2395 +/- ##
==========================================
+ Coverage 74.71% 74.86% +0.15%
==========================================
Files 110 111 +1
Lines 12105 12171 +66
==========================================
+ Hits 9044 9112 +68
+ Misses 2364 2358 -6
- Partials 697 701 +4 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your contribution!
Please check out this PR, which will be merged and rebase on this branch.
buf.gen.yaml
Outdated
@@ -1,6 +1,6 @@ | |||
version: v2 | |||
clean: true | |||
plugins: | |||
- remote: buf.build/protocolbuffers/go | |||
- remote: buf.build/protocolbuffers/go:v1.36.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we use the latest version?
- remote: buf.build/protocolbuffers/go:v1.36.0 | |
- remote: buf.build/protocolbuffers/go:v1.36.3 |
c3a2180
to
2798000
Compare
1a200ee
to
ba9f122
Compare
The
make generate-buf
command was using the latest version, which caused diffs in the generated files. To maintain consistency, the version was pinned to the one recently used to generate the p2p files.