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.

Sign up for GitHub

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

Jump to bottom

--version should not include log output #47

Closed
analog-cbarber opened this issue Jul 9, 2021 · 0 comments · Fixed by #64
Closed

--version should not include log output #47

analog-cbarber opened this issue Jul 9, 2021 · 0 comments · Fixed by #64

Comments

@analog-cbarber
Copy link

Currently, the --version flag includes logging output, e.g:

Log level set to ERROR
0.8.2+2.gd78be23

The log output is not useful in this case and hampers the ability for external tools to parse the version (to check for version compatibility).

Instead it should print the version and exit before initializing the logger.
analog-cbarber added a commit to analog-garage/conda-mirror that referenced this issue Jul 15, 2021
@analog-cbarber
--version does not log log level (conda-incubator#47)
202bed4 
Also do not log the log level if
verbosity is zero.
@xhochy xhochy mentioned this issue Feb 27, 2023
Merged
xhochy added a commit that referenced this issue Feb 27, 2023
@xhochy
Merge pull request #64 from conda-incubator/analog-garage/ag-misc
08899dd 
Fixes for issues #27, #31, #44, #47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes for issues #27, #31, #44, #47 conda-incubator/conda-mirror
1 participant
@analog-cbarber