k-wave-python icon indicating copy to clipboard operation
k-wave-python copied to clipboard

update logic to prevent duplicate workflows

Open waltsims opened this issue 2 years ago • 4 comments

waltsims avatar Dec 20 '23 19:12 waltsims

What is the aim of this PR?

faridyagubbayli avatar Jan 17 '24 08:01 faridyagubbayli

What is the aim of this PR?

We currently have duplicate CI workflows running in pull requests. The CI is triggered for pushed code and pull requests. This PR was aimed at ensuring the CI was triggered only once. It seems it's not so simple at first glance.

waltsims avatar Jan 17 '24 21:01 waltsims

Codecov Report

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

Project coverage is 67.97%. Comparing base (6c5ef8a) to head (f9a33ef). Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #235   +/-   ##
=======================================
  Coverage   67.97%   67.97%           
=======================================
  Files          48       48           
  Lines        7050     7050           
  Branches     1604     1604           
=======================================
  Hits         4792     4792           
- Misses       1684     1685    +1     
+ Partials      574      573    -1     
Flag Coverage Δ
3.10 68.16% <ø> (ø)
3.11 68.16% <ø> (ø)
3.12 68.16% <ø> (ø)
3.9 67.96% <ø> (ø)
ubuntu-latest 67.97% <ø> (ø)
windows-latest 67.97% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Feb 04 '24 03:02 codecov[bot]

This now only runs the ci on Pars. This logic makes sense since the master branch is protected.

waltsims avatar Feb 04 '24 04:02 waltsims