-
-
Notifications
You must be signed in to change notification settings - Fork 266
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
ci: add paths-ignore to workflows #6713
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## unstable #6713 +/- ##
=========================================
Coverage 61.87% 61.87%
=========================================
Files 557 562 +5
Lines 59209 59301 +92
Branches 1915 1916 +1
=========================================
+ Hits 36635 36695 +60
- Misses 22531 22563 +32
Partials 43 43 |
Performance Report✔️ no performance regression detected Full benchmark results
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might wanna come to a conclusion first in #6686 if we wanna use labels or ignore certain folders. Some requirements like skipping spec tests for certain PRs will be hard to achieve with this strategy.
We can definitely combine both approaches, but the question is if path based ignore is even required if we go with labels. If we go with this please make sure we are not skipping workflows that shouldn't be skipped. |
Some folders never require things like tests, so sounds like the best option for them. IMO
Sure yeah, that's why I went with the conservative approach of just ignoring |
Motivation
Do not trigger all workflows when irrelevant files are modified (e.g.
docs
).