-
Notifications
You must be signed in to change notification settings - Fork 220
feat: add workflow to test buf difs #3212
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
base: main
Are you sure you want to change the base?
Conversation
| with: | ||
| go-version-file: go.mod | ||
| - name: Set up buf | ||
| uses: bufbuild/buf-setup-action@v1 |
Check warning
Code scanning / CodeQL
Unpinned tag for a non-immutable Action in workflow Medium generated
Uses Step
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3212 +/- ##
==========================================
- Coverage 76.12% 76.08% -0.05%
==========================================
Files 310 310
Lines 29687 29687
==========================================
- Hits 22600 22588 -12
- Misses 5450 5456 +6
- Partials 1637 1643 +6 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| concurrency: | ||
| group: ${{ github.workflow }}-${{ github.ref }} | ||
| cancel-in-progress: true |
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.
We probably don't need this.
| make generate-buf | ||
| - name: Check for uncommitted changes | ||
| run: | | ||
| if ! git diff --exit-code; then |
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.
I think writing a new uncommitted file won't fail this.
I think I have an error somewhere because it should've failed either at the first or second commit error.
I think we can add a check for
go generatecode as well 🤔