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

add buf support to repo #126

Merged
merged 10 commits into from
Nov 30, 2022
Merged

add buf support to repo #126

merged 10 commits into from
Nov 30, 2022

Conversation

tac0turtle
Copy link
Member

@tac0turtle tac0turtle commented Nov 30, 2022

adds buf support and Golang/Rust generation script

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@codecov
Copy link

codecov bot commented Nov 30, 2022

Codecov Report

Base: 50.34% // Head: 38.11% // Decreases project coverage by -12.22% ⚠️

Coverage data is based on head (62fd300) compared to base (fea2d99).
Patch coverage: 51.61% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           master     #126       +/-   ##
===========================================
- Coverage   50.34%   38.11%   -12.23%     
===========================================
  Files          23        5       -18     
  Lines        7886     4182     -3704     
  Branches       85        0       -85     
===========================================
- Hits         3970     1594     -2376     
+ Misses       3560     2275     -1285     
+ Partials      356      313       -43     
Flag Coverage Δ
go 38.11% <51.61%> (ø)
rust ?
typescript ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
go/proofs.pb.go 31.51% <51.61%> (ø)
rust/src/cosmos.ics23.v1.rs
rust/src/lib.rs
rust/src/host_functions.rs
js/jasmine-testrunner.js
rust/src/compress.rs
js/src/testvectors.spec.ts
js/src/proofs.spec.ts
js/src/generated/codecimpl.js
js/src/testhelpers.spec.ts
... and 9 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Makefile Outdated
@$(protoImage) buf lint --error-format=json

proto-check-breaking:
@$(protoImage) buf breaking --against $(HTTPS_GIT)#branch=main
Copy link
Member

Choose a reason for hiding this comment

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

You need to set HTTPS_GIT as a variable and update the branch to master.

- uses: bufbuild/buf-push-action@v1
with:
input: "proto"
buf_token: ${{ secrets.BUF_TOKEN }}
Copy link
Member Author

@tac0turtle tac0turtle Nov 30, 2022

Choose a reason for hiding this comment

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

added token

Copy link
Member

@julienrbrt julienrbrt left a comment

Choose a reason for hiding this comment

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

tACK for Go and buf config.

@tac0turtle tac0turtle merged commit a21f341 into master Nov 30, 2022
@tac0turtle tac0turtle deleted the marko/buf branch November 30, 2022 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants