-
Notifications
You must be signed in to change notification settings - Fork 18
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
feat: add --exclude-core-addons flag #53
Conversation
@sbidoul CI is broken... 😅 |
--select-found
flag--select-found
and --exclude-core-addons
flags
One PR per feature, please. |
--select-found
and --exclude-core-addons
flags
OK, original PR is now in #54, and this one depends on it. I marked it as draft in the mean time. (I do it like this to avoid having to rewrite the test, and also because I need to deploy both patches and it's easier for me to have both in the same branch). |
Yeah, my point is that these features are logically independent, it's better to have a test for each individually. |
Codecov Report
@@ Coverage Diff @@
## main #53 +/- ##
==========================================
+ Coverage 83.77% 84.57% +0.79%
==========================================
Files 17 17
Lines 561 564 +3
Branches 114 115 +1
==========================================
+ Hits 470 477 +7
+ Misses 71 68 -3
+ Partials 20 19 -1
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
docs added |
Thanks! |
With this feature, together with
--select-found
, you can find all non-core addons automatically.@moduon MT-1075