- Notifications
You must be signed in to change notification settings - Fork744
Update ls with glob example.#1006
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
smack0007 commentedAug 28, 2020
I just added an example of using a glob pattern with ls. As a windows user this wasn't immediately obvious to me. |
smack0007 commentedAug 31, 2020
I took a look at the failed builds and I have no idea what to do. I only added a single comment to a function's description and regenerated the docs. Please advise. |
nfischer commentedSep 1, 2020
It looks like you did everything right, but maybe the test has become flaky. Can you trigger a rebuild of the PR? I can't find the "rebuild" button in the Travis UI anymore. The "brute force" way to trigger a rebuild is to |
nfischer left a comment
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.
This is otherwise fine. If you can't easily trigger a Travis rebuild, I'll look at merging this as-is.
codecov-commenter commentedSep 2, 2020 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Codecov Report
@@ Coverage Diff @@## master #1006 +/- ##======================================= Coverage 97.22% 97.22% ======================================= Files 35 35 Lines 1332 1332 ======================================= Hits 1295 1295 Misses 37 37
Continue to review full report at Codecov.
|
Adding example of using ls with globs.
Adding an example of using ls with globs. Will generate the README soon.