Skip to content
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

feat: Use GNU/POSIX-style argument syntax conventions for CLI flags #10

Merged
merged 1 commit into from
Jul 14, 2022

Conversation

timflannagan
Copy link
Collaborator

Signed-off-by: timflannagan [email protected]

@timflannagan
Copy link
Collaborator Author

@timflannagan
Copy link
Collaborator Author

Local output after these changes:

$ ./target/release/dcp --help
dcp 0.1.0
exdx
docker cp made easy

USAGE:
    dcp [FLAGS] [OPTIONS] <IMAGE>

FLAGS:
    -h, --help               Prints help information
    -V, --version            Prints version information
    -w, --write-to-stdout    Whether to write to stdout instead of the filesystem

OPTIONS:
    -p, --content-path <CONTENT-PATH>      Where in the container filesystem the content to extract is [default: /]
    -d, --download-path <DOWNLOAD-PATH>    Where the image contents should be saved on the filesystem [default: .]

ARGS:
    <IMAGE>    Container image to extract content from

@timflannagan
Copy link
Collaborator Author

whoops - need to update tests too

@timflannagan timflannagan force-pushed the feat/use-posix-style-cli-flags branch from 4db6085 to 5e6c851 Compare July 14, 2022 20:20
@timflannagan timflannagan force-pushed the feat/use-posix-style-cli-flags branch 2 times, most recently from aa47004 to bc4907e Compare July 14, 2022 20:25
tests/cli.rs Outdated Show resolved Hide resolved
@timflannagan timflannagan force-pushed the feat/use-posix-style-cli-flags branch from bc4907e to 5ab1964 Compare July 14, 2022 20:31
@timflannagan timflannagan merged commit b09da0f into exdx:main Jul 14, 2022
@timflannagan timflannagan deleted the feat/use-posix-style-cli-flags branch July 14, 2022 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants