[ci] Improve nightlies by dropping trigger workflow, fix naming-scheme - #91
[ci] Improve nightlies by dropping trigger workflow, fix naming-scheme#91aaronj0 wants to merge 2 commits into
Conversation
| @@ -1,4 +1,4 @@ | |||
| name: CI | |||
| name: Build & Test | |||
There was a problem hiding this comment.
Doesn’t that make the ci status row much longer?
There was a problem hiding this comment.
It should be a net shorten:
CI / macos-26-intel/llvm21/py3.14/c++20 / Build & Test
becomes
Build & Test / macos-26-intel/llvm21/py3.14/c++20
There was a problem hiding this comment.
Ah, ok, isn't the CI implying "Build & Test"?
There was a problem hiding this comment.
Yeah.. there is some redundancy there as it now shows it twice. The issue is the parent workflow in ciwf is named "Build & Test". Probably needs some change there. Do you want to drop "Build & Test"? I don't have any preference and would ideally like the smallest name with the relevant info without much verbosity/length
There was a problem hiding this comment.
I guess CI is a global term that is technically correct for any workflow, hence that's wouldn't differentiate testing from wheels and nightlies. We can probably have Test|Wheels|Nightlies as the category in front and the platform/arch/llvm/python as a consistent pattern that follows. How is that?
We would then have:
Test / macos-26-intel/llvm21/py3.14/c++20
Wheels / ubuntu-24.04/llvm22/py3.14/c++20
Nightly / ubuntu-24.04-arm/llvm21/py3.14/c++20
which to me is minimal in length with complete info
There was a problem hiding this comment.
I am fine as long as it is the shortest meaningful set which allows me to view these things on a reasonably-sized smart phone.
2 participants
Clean up CI naming scheme and drop redundant pr-nightly trigger. The mechanism works quite well when tested on open PR's (adding
build-wheelsortest-nightlyinstantly triggers extended runs for more coverage)