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
ls - Accept and process a list of dirs #735
base: master
Are you sure you want to change the base?
Conversation
else: | ||
for directory in where: | ||
rc.append(list_dir(directory)) | ||
return min(rc) |
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.
return success if at least one operation was a success.
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.
Might be better to have the above comment in the code rather than a long-lost PR
url += "?t=json" | ||
resp = do_http("GET", url) | ||
if resp.status == 404: | ||
print("%s: No such file or directory" % directory, file=stderr) |
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.
added dir/filename in the error.
dc54b99
to
6d3854c
Compare
Codecov Report
@@ Coverage Diff @@
## master #735 +/- ##
=======================================
- Coverage 92% 92% -0%
=======================================
Files 151 157 +6
Lines 27031 27491 +460
Branches 4044 4114 +70
=======================================
+ Hits 24955 25287 +332
- Misses 1432 1538 +106
- Partials 644 666 +22
Continue to review full report at Codecov.
|
8c69b1f
to
dae5e5b
Compare
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.
👍
…e dir returns a valid value
a fix for tahoe ls $THING $OTHERTHING` doesn't work
sample input/output