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
Ignore all node_modules #31
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.
Thanks! looks nice:)
@Raigen Thanks for the PR! Can you also change: https://github.com/babel/babel-upgrade/blob/master/src/index.js#L58? |
Ignore all occurrences of node_modules in possible subdirectories.
3477534
to
4ac2b77
Compare
@existentialism Done! |
lol I don't know how to use globs I guess? 👍 I am curious about this option though https://github.com/sindresorhus/globby#gitignore - would turning this on be the same affect? does anyone want to test that? |
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.
We can land this and deal with .gitignore stuff separately?
and we should figure out a test for this |
Ok made #32 |
Ignore all occurrences of node_modules in possible subdirectories.
Fixes #27