-
-
Notifications
You must be signed in to change notification settings - Fork 305
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
[New] Add --ignore-pattern
flag
#587
Conversation
This PR introduces the --ignore-pattern flag as a shorthand for ignoring test files. The flag may be used together with --ignore; the input will be concatenated in that case. I figured this behavior would make the most sense, since users may want to ignore everything in .gitignore and some other files on top. Fixes tape-testing#586
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.
Seems pretty reasonable, assuming the tests fail without the bin/tape changes.
651bb43
to
100f313
Compare
--ignore-pattern
flag
I've rebased this, and renamed the flag to All that's missing now is documentation. |
b3c4940
to
1e1d5fa
Compare
Codecov ReportBase: 96.36% // Head: 96.36% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #587 +/- ##
=======================================
Coverage 96.36% 96.36%
=======================================
Files 4 4
Lines 632 632
Branches 148 148
=======================================
Hits 609 609
Misses 23 23 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@ljharb cool, thank you 👌 I've added some documentation now, feel free to make any adjustments as needed! |
0de9ca9
to
8c9fe8e
Compare
This PR introduces the --ignorePattern flag as a shorthand for ignoring test files.
The flag may be used together with --ignore; the input will be concatenated in that case. I figured this behavior would make the most sense, since users may want to ignore everything in .gitignore and some other files on top.
Let me know if this looks good, then I'll also add some documentation :)
Fixes #586