Save minutes by avoiding duplicate test runs - Githubissues

iiasa / message_ix

The integrated assessment and energy systems model MESSAGEix

https://docs.messageix.org

Apache License 2.0

112 stars 149 forks source link

Save minutes by avoiding duplicate test runs #785

Closed glatterf42 closed 5 months ago

glatterf42 commented 5 months ago

This PR removes the trigger for CI runs on pushed to main to avoid duplicate test runs. This avoids duplicate runs.

In order for this to work properly, we should make sure that

[x] main can only be reached via PRs
[x] PRs need to be up to date with main
[x] PRs require several tests to be successful before merging

We might also want to

[x] expand the number of tests we require to be successful

All of which can be done via the settings -> branches.

For now, we require the py3.12 and tutorial checks for macos, ubuntu, and windows to be successful.

How to review

Read the diff and note that the CI checks all pass.

PR checklist

[x] Continuous integration checks all ✅
~[ ] Add or expand tests; coverage checks both ✅~ Just changing how often tests are run.
~[ ] Add, expand, or update documentation.~ Just changing how often tests are run.
~[ ] Update release notes.~ Just changing how often tests are run.

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (693f227) 95.2% compared to head (fd45f2f) 95.2%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #785 +/- ## ===================================== Coverage 95.2% 95.2% ===================================== Files 46 46 Lines 4335 4335 ===================================== Hits 4130 4130 Misses 205 205 ```