-
-
Notifications
You must be signed in to change notification settings - Fork 306
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 -i flag to specify ignore files #492
Conversation
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.
(I'll do a cleanup pass on dotignore
and publish a new patch if needed before merging this :-D)
Thanks, this is great!
73a6532
to
3aa123c
Compare
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.
Went ahead and made the changes. Thanks!
7b52a88
to
a0f9350
Compare
v4.13.0 - [New] `tape` binary: Add -i flag to ignore files from gitignore (#492) - [New] add `t.match()` and `t.doesNotMatch()`, new in `node` `v13.6` - [Deps] update `resolve` - [Tests] handle stack trace variation in node <= 0.8 - [lint] fix object key spacing
Sorry I didn't look at any of your comments and review. I am on the other side of the world as you. Timing issues 😝 |
This is a really weird feature. tape assumes git now and reads gitignore files? I am very against this. |
@substack are you only against the default of |
Fixes #428
Confirming to #428 (comment)
Closes #451.