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

Accept option to change the output folder name #8

Open
JerryHue opened this issue Oct 1, 2021 · 1 comment
Open

Accept option to change the output folder name #8

JerryHue opened this issue Oct 1, 2021 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@JerryHue
Copy link
Owner

JerryHue commented Oct 1, 2021

yassgy by default has the distribution folder to be named dist. While the user may rename the folder to their liking, it is a nice addition for yassgy to create a folder with a specified name.

We may accept the name when calling the program through the command line, by accepting an option labelled --output or -o. If there is no output folder name provided, dist will be used as default.

Thus, if we wrote

./yassgy -i input.txt -o out

then a directory called out should be created.

In the absence of a value when the option appears, an error has to be shown:

./yassgy -i input.txt -o

This means that the output option is required.

@JerryHue JerryHue added the enhancement New feature or request label Oct 1, 2021
@JerryHue JerryHue self-assigned this Oct 1, 2021
@JerryHue
Copy link
Owner Author

JerryHue commented Oct 2, 2021

Commit fd6255c addresses this.

However, right now this does not complete the requirements: an error message does not appear. Instead, the help message is shown.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant