-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
Add no-dir
flag
#5
Conversation
Hmm, doesn't seem to work fully since |
Should be fixed now. I think |
Remind me, why is this needed again? Seems like it should just be default behavior. |
@sindresorhus The problem: I'm working on @kevva s PR won't fix sindresorhus/cpy#10, but alleviates that problem, and because @sindresorhus Please also note #3 ("Added nodir option to cli.") of @chimon2000. |
Yeah, currently this module is unusable if you have files nested in directories since it'll error out when trying to read the dirs. |
Is there anything blocking this PR? This would fix: |
Bumping |
Thanks @kevva |
@danielweck, yes, they'll be ignored. |
hi guys! Sorry to bother but this PR seems to be gathering dust a little bit. What is needed to move it forward? Or should we just delete it? |
Closing in favor of #10 |
Related to sindresorhus/cpy#10. When copying all files and directories in a directory you'll end up with a
EISDIR
failure. And sincenodir
wasn't passed there was no way to fix it.