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

Fixed status checks on protected branches. #962

Merged
merged 1 commit into from
Mar 9, 2023

Conversation

MoFtZ
Copy link
Collaborator

@MoFtZ MoFtZ commented Mar 7, 2023

master is a protected branch in GitHub, and requires all-required-checks-done to pass.

As described in this article, when path filtering is used, other CI workflows need their own all-required-checks-done.

Added all-required-checks-done to website CI workflow. This is separate from the main CI workflow, so it needs its own.

Removed automated PR workflows from main CI ignore list. They will not be updated often, so not worth the trouble of a generic/fallback workflow as suggested in the article above. Otherwise, we would constantly need to update both the main CI workflow and fallback workflows for newly added paths to ignore.

@m4rs-mt m4rs-mt added the bug label Mar 9, 2023
@m4rs-mt m4rs-mt added this to the v1.4 milestone Mar 9, 2023
@MoFtZ MoFtZ merged commit b77cac9 into m4rs-mt:master Mar 9, 2023
@MoFtZ MoFtZ deleted the bug/ci-status-checks branch March 10, 2023 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants