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

Undocumented convention: The --reset flag must be the first argument #182

Open
nbars opened this issue Nov 14, 2022 · 0 comments
Open

Undocumented convention: The --reset flag must be the first argument #182

nbars opened this issue Nov 14, 2022 · 0 comments

Comments

@nbars
Copy link

nbars commented Nov 14, 2022

Hey,

I realized that --reset must be passed as the first argument since it is parsed by the entry point script of the container and not the script that parses everything else. It would be much nicer if the position would not be constrained by its position, or, at least, passing arguments considered unknown lead to a hard error. Since the error message --reset is an unknown argument is hard to spot because of the other output produced by the application.

Since this project seems to be unmaintained, I refrained from submitting a pull request. Please let me know if I am wrong, if so, I am happy to help if desired.

Best,
Nils

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

No branches or pull requests

1 participant