Skip to content

Commit 672e56c

Browse files
authored
chore(ci): Make "Required Tests passed" job depend on artifact upload and other required jobs (#8751)
Adjust our final "Required Tests Passed" CI job to not only depend on required _test_ jobs but on _all_ required jobs. Specifically, this adds * Lint * Circular deps check * Most importantly: Upload Artifacts With this change, we can configure craft's status provider to specifically wait for this job which should fix the artifacts download timeout (getsentry/craft#482). It's worth noting that this PR will only fix this timeout in our repo and in a way we're adjusting to the status provider check. However, given that we're apparently the only repo where this happens, it's probably justified and it makes things more explicit.
1 parent a4ae291 commit 672e56c

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

.github/workflows/build.yml

+5-2
Original file line numberDiff line numberDiff line change
@@ -855,8 +855,8 @@ jobs:
855855
timeout-minutes: 5
856856
run: yarn test:assert
857857

858-
job_required_tests:
859-
name: All required tests passed or skipped
858+
job_required_jobs_passed:
859+
name: All required jobs passed or were skipped
860860
needs:
861861
[
862862
job_build,
@@ -870,6 +870,9 @@ jobs:
870870
job_browser_loader_tests,
871871
job_remix_integration_tests,
872872
job_e2e_tests,
873+
job_artifacts,
874+
job_lint,
875+
job_circular_dep_check,
873876
]
874877
# Always run this, even if a dependent job failed
875878
if: always()

0 commit comments

Comments
 (0)