-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
showing dot files by default #2
Comments
wow thanks for the kind words! I spent way too much time on this lol. I've been using it a lot locally and I really love the API, I hope other people get some value from it!
ha @doowb mentioned something about this too. IMHO this should return dotfiles by default, and only exclude them when Thoughts? Thanks again! I was wondering how you were doing recently and was about to say hi, your ears must have been tingling! |
ME TOO!!! slowly moving it into my code base replacing other globs.
that's cool, no issues here. I was thrown off due to the readme file stating the default property for
yes they were! been meaning to contact you as well. just started coding again about 6 months ago. Actually it was @doowb's tweet on Twitter about VS Code that got me back into coding. So thank him for me. |
wow thanks!
good catch!
that's awesome! I was taking a break as well. Sounds like we both had some much needed time away. I'm sure @doowb will see this message! and it doesn't surprise me that he said something that got you coding again, he inspires me too! |
@akileez I added some examples to the readme in case it might come in handy. The examples show pretty much exactly how I personally use this lib. |
I think this is the case... the default should be const files = await readdir('.');
console.log(files);
//=> ['.DS_Store', '.git', 'LICENSE', 'README.md', 'package.json']
const files = await readdir('.', { dot: false });
console.log(files);
//=> ['LICENSE', 'README.md', 'package.json'] Then, this will also match the
This is awesome! I'm glad you're coding again :) |
thanks @jonschlinkert. already using isMatch, just didn't realize though. the last version it was called the and thanks again @doowb! ur the man! |
I'm so sorry, I know how lame this sounds but I looked at the dependents before I published and it looked like no one was using this besides me lol. So my brain wasn't in the the right place for writing a changelog. When you said you were using it, I should have updated the changelog! I'll get something put together, but I don't think there are that many changes besides what we talked about already. I just wanted to get the API tightened up.
looks perfect to me! |
no worries!!!! I usually copy/paste directly from github, so it wouldn't show up in any stats. I started watching because it was that good! The symlink change happened and I copied that version as well. This is the only way I picked up the change with If everything is fine, you are welcome to close this. |
readdir/lib/async.js
Line 64 in 4198053
should this be
||
as opposed to&&
or something else? (in lib/sync as well)after looking a little further, methinks the previous logic works due to options.dot never being initialized anywhere
btw, this is absolutely awesome!!!!!!!!!!!!!!!!!!!!!!
The text was updated successfully, but these errors were encountered: