-
Notifications
You must be signed in to change notification settings - Fork 93
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Stop running CI on nightly? #295
Comments
I disagree with this - it serves as an early warning of things that might change in the future and allows one to work with Graphs.jl also on nightly. The case, that some change is only temporary is committed to nightly that breaks this package but is then removed afterwards never as far as I know never occurred for any of the packages here. I also am not sure how this will hide breaking changes in the language itself - if that is indeed the case, then the developers of Julia should provide some kind of mechanism to disable the nightly tests on PkgEval. |
Fixed by making nightly CI failures non blocking |
I'd like to revisit this cause the signal sent by CI is very noisy due to nightly failures that are not always our fault. When you look at the PRs, they're all marked as failing because of nightly (the first three actually pass the tests). CI failures are not blocking for the badge on the README, but it's still a hassle for PR submitters and reviewers alike. |
I don't have a strong opinion on this because each time the test fails, I check if this is nightly or not, and it's not that annoying. On the other hand, I never dig to see what is failing on nightly, so nightly tests are kinda useless for me... |
True but it's an additional step, that may confuse infrequent contributors, and that makes it hard for us to quickly identify which PRs are ready and which are still failing just by looking at the list (since there's a red cross everywhere) |
It's not obvious at all that packages should run CI on nightly, for instance because it might hide actual breaking changes in the language itself. I suggest removing the nightly CI
See this discourse thread
The text was updated successfully, but these errors were encountered: