CI: Reduce the number of CI jobs - #108
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #108 +/- ##
=======================================
Coverage 78.27% 78.27%
=======================================
Files 34 34
Lines 925 925
=======================================
Hits 724 724
Misses 201 201 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
| # To reduce the number of CI jobs, we test Linux 64-bit against all Julia versions (see above), | ||
| # but for other platforms: |
There was a problem hiding this comment.
| # To reduce the number of CI jobs, we test Linux 64-bit against all Julia versions (see above), | |
| # but for other platforms: | |
| # We test Linux 64-bit against all Julia versions (see above), | |
| # but for other platforms we test less (see below). Thus reducing number of CI jobs |
oxinabox
left a comment
There was a problem hiding this comment.
I am not sure it's worth having CI on anything except linux if it doesn't test all versions.
Because a change that only effects one version could be made in any PR.
|
Should I just remove Windows and macOS CI, in that case? |
Yes, i think so |
|
I've removed all of the Windows and macOS CI jobs. What should we do about 32-bit Linux? |
Let's remove that too, and we can add it back in if someone finds that we have ever broken something on 32 bit |
|
I have to say, I don't think this is a good idea :) We should run CI on Windows, and we should run it on as many platforms/version combos as possible. This is a very low churn repo, so I don't really see the problem with running all these CI jobs occasionally. But the package has to work on all of these platforms, so we should test it on those. |
|
fair. |
Before this PR, we have 64 CI jobs, which is a lot.
This PR reduces the number of CI jobs:
After this PR, we have 23 CI jobs.