-
Notifications
You must be signed in to change notification settings - Fork 11
Open
Milestone
Description
Since dir
is a required option, I don't see the harm in making it the first argument (especially given the command name push-dir
. The extra dir
is redundant and I think actually harms readability and clarity)
push-dir --dir=build --branch=gh-pages
push-dir build --branch=gh-pages
Alternatively, I like this syntax:
push-dir build:gh-pages
Rationale:
- directory name is required
- branch name is required
- git branch names cannot contain
:
- mimicks
git push <remote_name> <local_branch_name>:<remote_branch_name>
syntax
The whole point of this module is to basically push the contents of a folder as if it were a branch, so I think it makes sense.
We could even make it symmetric so we have:
push-dir {optional_remote_name} <local_dir_name>:<remote_branch_name> {advanced_options}
Unfortunately, directory names can contain colons, so maybe this is alternative is better:
push-dir build gh-pages {advanced_options}
adeluccar
Metadata
Metadata
Assignees
Labels
No labels