On 11.08.22 00:18, Neil C Smith wrote:
On Wed, 10 Aug 2022 at 18:25, Michael Bien <[email protected]> wrote:
There are also additional command labels available, ci:all-tests would
enable everything, while ci:no-build would as you would guess disable
everything (e.g if all what the PR does is to edit a readme or sync-PRs
which syncs two branches could use this too, since they build everything
twice right now).
Really like these changes overall. However, I'm not sure what this
bit means - one of the reasons we use sync PRs is to run all CI tests
on them.
I haven't explained that very well sorry.
main.yaml our main workflow triggers on pull_request and push to the
main repo.
Usually push only happens on merge (since that counts as push too), if
you sync two branches on our main repo (not from clone to main), you
would push into one branch and create a PR from it. Which means it
builds the same hash twice just for the PR (and a third time on merge).
You would see every check twice below the sync PR, one marked with push
and one with pull_request. Unfortunately it doesn't show that on closed
PRs and we don't have one open atm.
So if you would set a sync PR to ci:no-build, it would ignore the PR
trigger but still build the push trigger -> it should hopefully only
build once. But sync PRs happen so rarely that it probably isn't worth
to optimize for that scenario. Setting it to ci:all-tests should be fine ;).
regards,
michael
Best wishes.
Neil
---------------------------------------------------------------------
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
---------------------------------------------------------------------
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
