CI, MAINT: Add docs-only option to CI#26316
Conversation
Borrows logic from SciPy CI Co-authored-by: Lucas Colley <lucas.colley8@gmail.com>
[skip azp][skip cirrus][skip actions]
| jobs: | ||
| get_commit_message: | ||
| name: Get commit message | ||
| uses: ./.github/workflows/commit_message.yml |
There was a problem hiding this comment.
This already skips the run if not docs-only, right? Can we encode that into the name?
In general, I think having [docs only] is awesome (a github label might also be, easier to add by maintainer, but a different thing).
There was a problem hiding this comment.
Agreed, the naming in SciPy is historical as an existing job was modified.
There was a problem hiding this comment.
Thought I would ping since I just saw a suggestion to add the three skips we currently need.
Not sure if there is anything else holding it up. But, assuming it works, from my side it seems ready with that rename to clarify what the needs: does below.
|
@melissawm ping |
|
Sorry for the delay - these are not working in my fork which is why I'm holding it up. Will try to test again. |
|
ping @melissawm |
Borrows logic from SciPy CI skip checks implemented by @lucascolley
I will test on my own fork first before marking as ready for review, but if you have corrections or suggestions already please let me know.
See #26311 (comment)