Skip to content

Commit 7e32f9e

Browse files
committed
chore(all|pr_compliance): remove duplicate pre-commit; update needs
[What] Remove duplicated pre-commit invocation from main workflow and make platform jobs depend directly on PR Compliance via needs. [Why] Avoid running pre-commit twice on PRs and ensure all jobs are strictly gated by PR Compliance. [How] - .github/workflows/main.yml: drop pre-commit job; set ubuntu/mac/windows to needs: [pr_compliance]. Scope: - Task: 0 - Variant: 0 - Technology: all - Folder: pr_compliance Tests: - CI: workflow graph shows no duplicate pre-commit; downstream jobs start only after pr_compliance. Local runs: - N/A (workflow change only)
1 parent 00d90d9 commit 7e32f9e

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

.github/workflows/main.yml

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,20 +22,17 @@ jobs:
2222
pr_compliance:
2323
uses: ./.github/workflows/pr-compliance.yml
2424

25-
pre-commit:
26-
needs: [pr_compliance]
27-
uses: ./.github/workflows/pre-commit.yml
2825
ubuntu:
2926
needs:
30-
- pre-commit
27+
- pr_compliance
3128
uses: ./.github/workflows/ubuntu.yml
3229
mac:
3330
needs:
34-
- pre-commit
31+
- pr_compliance
3532
uses: ./.github/workflows/mac.yml
3633
windows:
3734
needs:
38-
- pre-commit
35+
- pr_compliance
3936
uses: ./.github/workflows/windows.yml
4037
perf:
4138
needs:

0 commit comments

Comments
 (0)