Skip to content

Commit cddce26

Browse files
Pigbibicodex
andcommitted
fix(ci): gate drift on successful preflight
Co-Authored-By: Codex <noreply@openai.com>
1 parent da3de79 commit cddce26

2 files changed

Lines changed: 1 addition & 4 deletions

File tree

.github/workflows/drift-check.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ jobs:
2020
timeout-minutes: 15
2121
permissions:
2222
contents: read
23+
id-token: write
2324
env:
2425
FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: "true"
2526
LIFECYCLE_PERFORMANCE_BUCKET: ${{ vars.LIFECYCLE_PERFORMANCE_BUCKET || '' }}
@@ -115,7 +116,6 @@ jobs:
115116
PY
116117
117118
drift:
118-
if: ${{ always() && (needs.preflight_backtests.result == 'success' || needs.preflight_backtests.result == 'skipped') }}
119119
needs: preflight_backtests
120120
permissions:
121121
contents: read

tests/test_drift_workflow_config.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,8 @@ def test_drift_workflow_wires_pipeline_repo_and_lifecycle_env() -> None:
1717
assert "LIFECYCLE_PREFLIGHT_STAGING_ROOT" in workflow
1818
assert "Promote staged lifecycle backtests" in workflow
1919
assert "github.ref == format('refs/heads/{0}', github.event.repository.default_branch)" in workflow
20-
assert "needs.preflight_backtests.result == 'skipped'" in workflow
2120
assert "head.repo.full_name == github.repository" in workflow
2221
assert "id-token: write" in workflow
23-
preflight = workflow.split(" drift:", maxsplit=1)[0]
24-
assert "id-token: write" not in preflight
2522
assert "uses: QuantStrategyLab/QuantPlatformKit/.github/workflows/reusable-drift-check.yml@17278db4e7aef0007346d853eb308b6c1bd8c859" in workflow
2623
assert "strategy_domain: crypto" in workflow
2724
assert "caller_event_name: ${{ github.event_name }}" in workflow

0 commit comments

Comments
 (0)