Skip to content

CI Deadlock fix - #24154

Merged
Aragas merged 1 commit into
masterfrom
cherry-pick/pr-24147-ci-required-workflow-fix-to-master
Sep 9, 2026
Merged

Aragas merged 1 commit into
masterfrom
cherry-pick/pr-24147-ci-required-workflow-fix-to-master

Conversation

@nexusmodsvortex

Copy link
Copy Markdown

Cherry-pick of #24147 into master.


Fixed a deadlock where we require the main.yml workflow to be run and be successful and paths-ignore actually skipping the workflow execution

We now check whether the workflow should be skipped inside the workflow execution.

See https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/collaborating-on-repositories-with-code-quality-features/troubleshooting-required-status-checks#handling-skipped-but-required-checks

Avoid requiring workflows that can be skipped.

Since we can't skip it, lets just mark it as successful 🤷‍♂️

@nexusmodsvortex
nexusmodsvortex Bot requested a review from a team as a code owner September 9, 2026 14:05
@Aragas
Aragas enabled auto-merge September 9, 2026 14:05
@Aragas
Aragas merged commit ca92ee9 into master Sep 9, 2026
9 checks passed
@Aragas
Aragas deleted the cherry-pick/pr-24147-ci-required-workflow-fix-to-master branch September 9, 2026 14:20
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.

1 participant