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
Update ignore
to v5
#6975
Update ignore
to v5
#6975
Conversation
9522318
to
24879bd
Compare
src/cli/util.js
Outdated
try { | ||
fileIgnored = ignorer.ignores(path.relative(process.cwd(), filename)); | ||
} catch (_) { | ||
// Invalid path |
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.
Would it be better to do this update in next
instead and show this error to the user? I’m not sure what an “invalid path” is, but it sounds useful to get to know that some of my patterns are wrong?
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 don't know. I'm not changing current behave
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.
@evilebottnawi Do you have experience from ESLint or stylelint?
…nore # Conflicts: # yarn.lock
Co-Authored-By: Simon Lydell <simon.lydell@gmail.com>
try { | ||
fileIgnored = ignorer.ignores(path.relative(process.cwd(), filename)); | ||
} catch (_) { | ||
// Invalid path. TODO: Should we show this error to the user, rather than swallowing it? |
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.
Maybe we should add note for next major release about that?
/cc @fisker what is status? |
Not sure what I can do |
After reading sindresorhus/globby#120 , sindresorhus/globby#112 and sindresorhus/globby#104 I think I did something wrong |
docs/
directory)changelog_unreleased/*/pr-XXXX.md
file followingchangelog_unreleased/TEMPLATE.md
.✨Try the playground for this PR✨