-
Notifications
You must be signed in to change notification settings - Fork 232
Add ls option #616
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 ls option #616
Conversation
Codecov Report
@@ Coverage Diff @@
## master #616 +/- ##
=======================================
Coverage 78.32% 78.32%
=======================================
Files 5 5
Lines 798 798
Branches 232 232
=======================================
Hits 625 625
Misses 115 115
Partials 58 58
Continue to review full report at Codecov.
|
@pythops Do you want to make your own pull request / redo the branch ? (So you get full credit for this?) I can wait as long as you'd like! No hurry! |
@tony Just merge it :) |
@pythops Merged! Thank you for the contribution and the tests! I updated tests and switched it to use glob (since it's easier on package maintainers downstream, |
@tony glad to contribute :) |
#599 rebased