Skip to content

Conversation

@rodrigo-pino
Copy link
Collaborator

@rodrigo-pino rodrigo-pino commented Oct 24, 2025

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 generate code as well 🤔

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

Unpinned 3rd party Action 'Verify generated protobufs' step
Uses Step
uses 'bufbuild/buf-setup-action' with ref 'v1', not a pinned commit hash
@codecov
Copy link

codecov bot commented Oct 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.08%. Comparing base (02716f0) to head (66a5b72).

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.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment on lines +12 to +14
concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true
Copy link
Contributor

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
Copy link
Contributor

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.

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.

3 participants