tape-testing / tape

tap-producing test harness for node and browsers
MIT License
5.77k stars 307 forks source link

[New] Add `--ignore-pattern` flag #587

Closed ppati000 closed 1 year ago

ppati000 commented 1 year ago

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 https://github.com/ljharb/tape/issues/586

ljharb commented 1 year ago

I've rebased this, and renamed the flag to ignore-pattern to more closely match command-line convention.

All that's missing now is documentation.

codecov[bot] commented 1 year ago

Codecov Report

Base: 96.36% // Head: 96.36% // No change to project coverage :thumbsup:

Coverage data is based on head (8c9fe8e) compared to base (e9c9aba). Patch has no changes to coverable lines.

Additional details and impacted files ```diff @@ 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](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jordan+Harband). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jordan+Harband)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

ppati000 commented 1 year ago

@ljharb cool, thank you 👌 I've added some documentation now, feel free to make any adjustments as needed!