Skip to content

Skip tests action if only other actions changed #5976

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

Merged
merged 1 commit into from
Jul 17, 2022
Merged

Conversation

michaelosthege
Copy link
Member

The tests don't need to run if it's just other workflows that are being changed.

What is this PR about?
Reducing the number of unnecessary tests.yml workflows.

This is mostly in preparation of working on #5974, which will trigger a ton of CI.

The change needs to be made beforehand (I tried to do it in #5975), because the diff is evaluated between the branches, not just the commits.

Checklist

Major / Breaking Changes

None

Bugfixes / New features

None

Docs / Maintenance

Updated trigger conditions for the tests GH action to reduce the number of unnecessary trigger events.

The tests don't need to run if it's just other workflows that are being changed.
@michaelosthege michaelosthege added Github CI/CD no releasenotes Skipped in automatic release notes generation labels Jul 15, 2022
@michaelosthege michaelosthege requested a review from twiecki July 15, 2022 11:03
@michaelosthege michaelosthege self-assigned this Jul 15, 2022
@codecov
Copy link

codecov bot commented Jul 15, 2022

Codecov Report

Merging #5976 (a94b681) into main (2583b7f) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #5976      +/-   ##
==========================================
+ Coverage   89.35%   89.38%   +0.02%     
==========================================
  Files          73       73              
  Lines       13251    13251              
==========================================
+ Hits        11841    11844       +3     
+ Misses       1410     1407       -3     
Impacted Files Coverage Δ
pymc/parallel_sampling.py 86.79% <0.00%> (+0.99%) ⬆️

@twiecki twiecki merged commit 78da937 into main Jul 17, 2022
@twiecki twiecki deleted the fewer-useless-tests branch July 17, 2022 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Github CI/CD no releasenotes Skipped in automatic release notes generation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants