-
Notifications
You must be signed in to change notification settings - Fork 628
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: update protobuf generation to latest buf changes #2705
Merged
Merged
Changes from all commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
75738e4
updating buf proto generation setup
damiannolan 0db3744
revert adding test msg type
damiannolan 478dcd0
updating proto-swagger-gen configs
damiannolan 4c1d3e9
Merge branch 'main' into damian/buf-update
damiannolan db3fe49
Merge branch 'main' into damian/buf-update
damiannolan 8440402
updating to latest proto builder image
damiannolan fa3651b
downgrade to 0.46 support
damiannolan 26e3868
go mod tidy e2e
damiannolan 120024e
cleanup makefile
damiannolan 61b2aaf
rerun make proto-update-deps
damiannolan 3f82932
updating buf yaml
damiannolan 8f840c0
updating cosmos/gogo-proto pin
damiannolan 20ee1d6
regenerate protos
damiannolan d21ad24
go mod tidy e2e
damiannolan e3f51da
removing pulsar buf yaml
damiannolan 3781043
Update proto/buf.yaml
damiannolan 6deb01e
adding proto-registry.yml workflow for buf push to BSR
damiannolan 8fb165a
Merge branch 'damian/buf-update' of github.com:cosmos/ibc-go into dam…
damiannolan File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
name: Buf-Push | ||
# Protobuf runs buf (https://buf.build/) push updated proto files to https://buf.build/cosmos/ibc | ||
# This workflow is only run when a .proto file has been changed | ||
on: | ||
push: | ||
branches: | ||
- main | ||
paths: | ||
- "proto/**" | ||
|
||
jobs: | ||
push: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v3 | ||
- uses: bufbuild/[email protected] | ||
- uses: bufbuild/buf-push-action@v1 | ||
with: | ||
input: "proto" | ||
buf_token: ${{ secrets.BUF_TOKEN }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,5 +5,4 @@ | |
# previous "buf.yaml" configuration. | ||
version: v1 | ||
directories: | ||
- proto | ||
- third_party/proto | ||
- proto |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
todo:(for updating to sdk v0.47 we need)