mbien commented on code in PR #6630:
URL: https://github.com/apache/netbeans/pull/6630#discussion_r1827093434
##########
.github/workflows/main.yml:
##########
@@ -2606,13 +2612,14 @@ jobs:
env "netbeans.extra.options=-J-Dnetbeans.logger.console=true" ant
$OPTS test-vscode-ext
-# last job depends on everything so that it is forced to run last even if a
long job fails early
+# cleanup job depends on everything so that it is forced to run last even if a
long job fails early.
+# 'paperwork' is left out intentionally, since it doesn't run unit tests
(hopefully doesn't need restarts)
+# and shouldn't prevent cleanup on validation failure - which might be common
during dev time
cleanup:
name: Cleanup Workflow Artifacts
needs:
- base-build
- commit-validation
- - paperwork
Review Comment:
this is new: the paperwork job will no longer prevent CI pipeline artifact
cleanup. This could otherwise create a situation where workspaces are never
cleaned up (due to failing paperwork job) until shortly before PR integration -
which wouldn't be ideal.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
