-
Notifications
You must be signed in to change notification settings - Fork 379
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
#281 - Multi-file put/get for smb #282
Conversation
Thanks for the PR man! I'll test and update this, the nargs will need to be changed at least, but I'll fix it up. Appreciate the initial work on it! |
Shouldn't need to change
Thought it seems like my |
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.
LGTM!
I was originally thinking that you'd pass in pairs into a single --put-file/etc, which is why I figured the nargs would be changed, but this works better I think.
@wumb0 if you got the time feel free to add the changes to the wiki :) |
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.
LGTM
#281
I did not test this :)
Can do folder put later if you want.