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

Remove allowance for failure in PR checks #7997

Closed
wants to merge 2 commits into from
Closed

Remove allowance for failure in PR checks #7997

wants to merge 2 commits into from

Conversation

drew2a
Copy link
Collaborator

@drew2a drew2a commented Apr 22, 2024

The code change removes the option that allowed failures in pull request checks. This means all checks must now pass successfully.

Fixes #7995

Reference:

The code change removes the option that allowed failures in pull request checks. This means all checks must now pass successfully, ensuring a higher standard of code quality before merging.
Added an option to allow failure in the pull request checks. This change makes it possible for a check to fail without causing the entire workflow to fail, thus improving the robustness of our CI/CD pipeline.
@drew2a drew2a closed this Apr 22, 2024
@drew2a drew2a deleted the fix/7995 branch April 22, 2024 13:38
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.

Gatekeeper.yml gatekeeper Label check failed: required 1 of 'PR: safe to check', but found 0.
1 participant