On 16.08.22 17:23, Michael Bien wrote:
On 16.08.22 12:20, Neil C Smith wrote:
On Tue, 16 Aug 2022 at 09:25, Neil C Smith <[email protected]>
wrote:
On Thu, 11 Aug 2022 at 14:20, Michael Bien <[email protected]> wrote:
The branch tests run on the head of the delivery branch
(refs/head/delivery). The sync PR tests run on the pending result of
the merge to the base branch (refs/pull/<ID>/merge), eg. master.
The above assumption might need checking. Just restarting a
pull-request test on the sync PRs and noticed that it restarted in
both - ie. two sync PRs with different base branches share a single
set of tests on both GitHub and Travis. At the moment, it's the
result of the merge to master, so the important one. Not sure how
consistent the behaviour is - workflow history shows the other one
cancelled.
OK, this is probably due to
https://github.com/apache/netbeans/pull/3932 I guess we need to group
by head and base ref?
yeahhh... maybe. This is making my head smoke thinking about all
scenarios this workflow has to deal with.
Played with it a bit in a private repo and simply concatenating the
base_ref might indeed work (even without having a ternary operator):
group: ${{ github.head_ref || github.run_id }}-${{ github.base_ref }}
non PR:
"1234-"
PRs:
"head_ref-base_ref"
-> two sync PRs syncing two distinct branches won't cancel each other
anymore.
-mbien
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
