Skip to content
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

Modify Cirrus CI config to save resources #735

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

abravalheri
Copy link
Collaborator

Purpose

Recently Cirrus CI changed its free limits for open source. The idea behind this PR is to minimise the usage via a "fail fast" approach.

Approach

The changes can be summarised as follows:

  1. Run "build" and "check" inside the same container, so we save the time to spawn a separated container.
  2. Decrease the amount of Linux containers used. I left the lower and upper limits for the Python version in PR builds and added an intermediary version to the master/main branch.
  3. Wait until the Linux tests pass before running more expensive ones (Windows, OS X, FreeBSD).

Resources & Links

https://cirrus-ci.org/blog/2023/07/17/limiting-free-usage-of-cirrus-ci/

Recently Cirrus CI changed its free limits for open source.
The idea behind this PR is to minimise the usage via a "fail fast"
approach.

The changes can be summarised as follows:

1. Run "build" and "check" inside the same container, so we save the
   time to spawn a separated container.
2. Decrease the amount of Linux containers used.
   I left the lower and upper limits for the Python version in PR builds
   and added an intermediary version to the master/main branch.
3. Wait until the Linux tests pass before running more expensive ones
   (Windows, OS X, FreeBSD).
@coveralls
Copy link

coveralls commented Oct 25, 2023

Coverage Status

coverage: 97.907% (-0.03%) from 97.936% when pulling 473f9b5 on cirrusci into 14ff855 on master.

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.

None yet

2 participants