iiasa / ixmp

The ix modeling platform for integrated and cross-cutting scenario analysis
https://docs.messageix.org/ixmp
Apache License 2.0
36 stars 110 forks source link

Save minutes by avoiding duplicate test runs #515

Closed glatterf42 closed 2 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

We might also want to

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

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

How to review

PR checklist

codecov[bot] commented 5 months ago

Codecov Report

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

Project coverage is 98.9%. Comparing base (4119b93) to head (c3a7153).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #515 +/- ## ===================================== Coverage 98.9% 98.9% ===================================== Files 44 44 Lines 4804 4804 ===================================== Hits 4754 4754 Misses 50 50 ```
khaeru commented 5 months ago

Some notes for this and the parallel PRs iiasa/message_ix#785, iiasa/message-ix-models#150, iiasa/message_data#527:

khaeru commented 2 months ago

@glatterf42 should we merge this? Anything else to be done?

glatterf42 commented 2 months ago

I think it's good to go :)