-
-
Notifications
You must be signed in to change notification settings - Fork 383
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
Added command: LOAD #2142
Added command: LOAD #2142
Conversation
Could this be used to load URLs and MBIDs as well? If so you might be better off passing the argument list to |
My thinking was that it should operate the same as files, etc. entered on the command line, as described by @zas in his comment on #2137.
|
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
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.
Looks fine to me.
Summary
A command that allows to load files as a part of the command pipeline
Problem
It would be useful to load a file after some commands have already been executed as a part of the pipeline. The
load
command extends positional arguments' functionality.Solution
Added the needed command
Action
Should be squashed on merge