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

[feature] Implement verbosity levels for 'remote' command #17014

Open
1 task done
cubanpit opened this issue Sep 18, 2024 · 3 comments
Open
1 task done

[feature] Implement verbosity levels for 'remote' command #17014

cubanpit opened this issue Sep 18, 2024 · 3 comments
Assignees

Comments

@cubanpit
Copy link

What is your suggestion?

As far as I can tell, when running conan remote commands the output does not depend on the verbosity level.
I tried setting it from command line and conan remote -vquiet disable '*' is not considered a valid command, other variations just ignore the setting and print the same output.
I tried setting it from environment variable CONAN_LOG_LEVEL, but it is again ignored.
From a quick look at the code I would guess it is not implemented at the moment, I think it would be nice to have.

Have you read the CONTRIBUTING guide?

  • I've read the CONTRIBUTING guide
@memsharded memsharded self-assigned this Sep 18, 2024
@memsharded
Copy link
Member

Hi @cubanpit

Thanks for your suggestion.

I tried setting it from command line and conan remote -vquiet disable '*' is not considered a valid command, other variations just ignore the setting and print the same output.

I'd like to understand what would be the problem we would be trying to address here. Yes, it is true that at the moment the remote commands doesn't implement different levels of verbosity, but this is because their output is always very reduced, so what would be the benefit of conan remote disable not reporting what remotes were actually disabled?

@cubanpit
Copy link
Author

I would expect consistency with what the verbose options are saying, for example -werror would only print errors, the listing of the disabled remotes is not an error.
Of course this is only a minor detail, but it is nice to be able to set a policy and know that all Conan commands are respecting it.

@memsharded
Copy link
Member

Sure, sure, I agree that things should be consistent, I was trying to understand the "magnitude" of the issue and trying to learn if there was there some other use case, flow, or anything that we could be missing, that would be affected by this, also to be able to prioritize the work.

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

2 participants