-
Notifications
You must be signed in to change notification settings - Fork 9
ci: restrict codecov workflows to merge_group and workflow_dispatch #513
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
Conversation
emhane
left a comment
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.
lgtm
|
pending @dhyaniarun1993 |
513fb00 to
45488f4
Compare
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.
at second look, we defo want to revert the changes to this file
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.
Oh mb, done !
Remove pull_request and push triggers from coverage workflows to reduce CI resource usage. Codecov will now only run in merge queue or when manually triggered. Closes paradigmxyz#500
98351cd to
f1d803f
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
|
@emhane I think it's ready now. |
Summary
pushandpull_requesttriggers from coverage workflowsmerge_grouptrigger to run codecov only in merge queueworkflow_dispatchfor manual triggering when neededThis reduces CI resource usage by running codecov only when necessary.
Closes #500