-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Verify generated files are up to date in merge check #3007
Conversation
Codecov ReportBase: 54.02% // Head: 54.02% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #3007 +/- ##
=======================================
Coverage 54.02% 54.02%
=======================================
Files 144 144
Lines 8311 8311
=======================================
Hits 4490 4490
Misses 3493 3493
Partials 328 328 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. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: johngmyers, kishorj The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Issue
N/A
Description
Adds to the CI checks a verification that files generated by
make generated
are up-to-date.An example of a failure is in #3006.
Checklist
README.md
, or thedocs
directory)BONUS POINTS checklist: complete for good vibes and maybe prizes?! 🤯