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

8862 Version and Help cmd errors #8875

Merged
merged 1 commit into from Apr 29, 2024

Conversation

par456
Copy link
Collaborator

@par456 par456 commented Apr 29, 2024

Resolves #8862

We use the CommandLineParser nuget package to handle command line arguments. It currently has a bug with the help and version auto-arguments that causes it to always print an error when those are used.
commandlineparser/commandline#630

I've basically done the same thing as suggested in that issue which is to just not print out the error on those two arguments. They shouldn't throw an error anyway and do still work.

@par456 par456 requested a review from hol353 April 29, 2024 06:17
@hol353 hol353 merged commit 3dfa4ac into APSIMInitiative:master Apr 29, 2024
2 checks passed
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.

When running apsim.exe --version from CLI, I get an error
2 participants