Skip to content

CI: Reduce the number of CI jobs#108

Open
DilumAluthge wants to merge 2 commits intomainfrom
dpa/ci
Open

CI: Reduce the number of CI jobs#108
DilumAluthge wants to merge 2 commits intomainfrom
dpa/ci

Conversation

@DilumAluthge
Copy link
Member

@DilumAluthge DilumAluthge commented Dec 13, 2025

Before this PR, we have 64 CI jobs, which is a lot.

This PR reduces the number of CI jobs:

  1. For Linux 64-bit, we test against every Julia version.
  2. For Linux 32-bit, we only test against Julia v1.
  3. For Windows 64-bit, we only test against Julia LTS and Julia v1.
  4. For Windows 32-bit, we only test against Julia v1.

After this PR, we have 23 CI jobs.

@DilumAluthge DilumAluthge marked this pull request as ready for review December 13, 2025 20:28
@codecov
Copy link

codecov bot commented Dec 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.27%. Comparing base (51b54a7) to head (8ab5789).

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 Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment on lines +58 to +59
# To reduce the number of CI jobs, we test Linux 64-bit against all Julia versions (see above),
# but for other platforms:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# 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

Copy link
Member

@oxinabox oxinabox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@DilumAluthge
Copy link
Member Author

Should I just remove Windows and macOS CI, in that case?

@oxinabox
Copy link
Member

Should I just remove Windows and macOS CI, in that case?

Yes, i think so

@DilumAluthge
Copy link
Member Author

I've removed all of the Windows and macOS CI jobs.

What should we do about 32-bit Linux?

@oxinabox
Copy link
Member

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

Copy link
Member

@oxinabox oxinabox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merge when you are happy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants