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

Do not fail when non-critical workflow steps fail #1015

Merged
merged 1 commit into from
Aug 29, 2023

Conversation

dkfellows
Copy link
Member

Some workflow stages are non-critical. They can fail for various reasons (such as complexities of credentials for bots) and yet aren't actually very important if they don't run successfully for a particular workflow job. This marks them as something that the workflow can keep going after. (Coverage reporting already was such a stage.)

This was causing dependabot-submitted jobs to fail (e.g., #1012).

Some workflow stages are non-critical. They can fail for various reasons
(such as complexities of credentials for bots) and yet aren't actually
very important if they don't run successfully for a particular workflow
job. This marks them as something that the workflow can keep going
after. (Coverage reporting already was such a stage.)

This was causing dependabot-submitted jobs to fail.
@dkfellows dkfellows added bug Something isn't working minor Low priority blocker Do not release when one of these is open! labels Aug 29, 2023
@dkfellows dkfellows added this to the 7.1 milestone Aug 29, 2023
@dkfellows dkfellows self-assigned this Aug 29, 2023
@coveralls
Copy link

Coverage Status

coverage: 36.875% (-0.06%) from 36.939% when pulling d4dd1fb on no-fail-opt-steps into 3aaa0af on master.

@dkfellows dkfellows added this pull request to the merge queue Aug 29, 2023
Merged via the queue into master with commit 37cd933 Aug 29, 2023
9 checks passed
@dkfellows dkfellows deleted the no-fail-opt-steps branch August 29, 2023 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocker Do not release when one of these is open! bug Something isn't working minor Low priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants