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

Use named arguments #2

Closed
AlexWinder opened this issue Jan 16, 2022 · 0 comments · Fixed by #3
Closed

Use named arguments #2

AlexWinder opened this issue Jan 16, 2022 · 0 comments · Fixed by #3
Labels
enhancement New feature or request

Comments

@AlexWinder
Copy link
Owner

AlexWinder commented Jan 16, 2022

Currently if you want to customise the script for different system it requires manually changing the backup.sh script. This works fine however if the script is ever updated to include new features then those who have manually changed their script will need to adjust their changes to meet the new version.

Making use of named arguments would resolve this and allow for certain settings to be defined when the script is executed, for example:

./backup.sh --from /etc/letsencrypt --to /var/backups/letsencrypt --keep-for 120

A --help flag should be added to list all of the definitions.

@AlexWinder AlexWinder added the enhancement New feature or request label Jan 16, 2022
@AlexWinder AlexWinder changed the title Use named parameters Use named arguments Jan 16, 2022
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

Successfully merging a pull request may close this issue.

1 participant