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

Fix ignored workflow logic #6096

Merged
merged 1 commit into from
Sep 20, 2023
Merged

Conversation

svenklemm
Copy link
Member

@svenklemm svenklemm commented Sep 19, 2023

The paths filter in github workflows will trigger when at least
one of the pathes match unless everything else has been explicitly
excluded. For the ignored workflows we want it to only trigger
when only those files explicitly specified are changed and nothing
else.

Disable-check: force-changelog-file

@github-actions
Copy link

@antekresic, @akuzm: please review this pull request.

Powered by pull-review

@svenklemm svenklemm force-pushed the ignored_workflow branch 2 times, most recently from 557688e to 01b3725 Compare September 19, 2023 21:45
The paths filter in github workflows will trigger when at least
one of the pathes match unless everything else has been explicitly
excluded. For the ignored workflows we want it to only trigger
when only those files explicitly specified are changed and nothing
else.
@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Merging #6096 (228c395) into main (646cecd) will decrease coverage by 0.06%.
Report is 11 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #6096      +/-   ##
==========================================
- Coverage   81.53%   81.48%   -0.06%     
==========================================
  Files         246      246              
  Lines       56708    56727      +19     
  Branches    12571    12568       -3     
==========================================
- Hits        46238    46225      -13     
+ Misses       8098     8093       -5     
- Partials     2372     2409      +37     

see 48 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@svenklemm svenklemm requested a review from konskov September 20, 2023 07:05
@svenklemm svenklemm enabled auto-merge (rebase) September 20, 2023 07:07
@svenklemm svenklemm merged commit 9dc699a into timescale:main Sep 20, 2023
35 of 36 checks passed
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.

3 participants