scikit-image
Screen for sphinx warnings in nightly CI jobs
Description
... instead of on Azure. I feel like this is the better place to catch warnings early.
The failure addressed by https://github.com/scikit-image/scikit-image/pull/7226 prompted this change.
Checklist
- A descriptive but concise pull request title
- Docstrings for all functions
- Unit tests
- A gallery example in
./doc/examplesfor new features - Contribution guide is followed
Release note
Summarize the introduced changes in the code block below in one or a few sentences. The summary will be included in the next release notes automatically:
...
I think this would be an improvement. But who gets notified of nightly failures, and how?
But who gets notified of nightly failures, and how?
The "Test Nightly" job is triggered on pushes to main. So the core dev creating the merge should be notified explicitly. And a failure should be somewhat visible on our main GitHub page.
It's not ideal but I think we discussed this problem somewhere earlier and found this to be the best solution. We could look into creating an issue automatically on failure?