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

Consider using -v flag for neofs-cli #765

Open
carpawell opened this issue Mar 12, 2024 · 3 comments
Open

Consider using -v flag for neofs-cli #765

carpawell opened this issue Mar 12, 2024 · 3 comments
Labels
enhancement Improving existing functionality good first issue Good for newcomers I3 Minimal impact S3 Minimally significant U2 Seriously planned

Comments

@carpawell
Copy link
Member

carpawell commented Mar 12, 2024

That is verbose for cli. We have it but almost no one uses it and that is why it may be bad. Seems like tests are the place where it may play well. If we have some feedback, we can improve it.

Current Behavior

No steps use -v flag.

Expected Behavior

Use -v flag cause tests are expected to tell us as much info as possible when something is broken.

Possible Solution

Use -v flag. Maybe it can be optional.

Steps to Reproduce

Run tests, see no -v flag.

Context

Recently I really had to push debug prints to the PR to know what was the reason for the failure I faced. -v would answer me immediately.

@carpawell carpawell added the U2 Seriously planned label Mar 12, 2024
@roman-khimov roman-khimov added enhancement Improving existing functionality S3 Minimally significant I3 Minimal impact labels Mar 12, 2024
@roman-khimov
Copy link
Member

@evgeniiz321, thoughts?

@evgeniiz321
Copy link
Contributor

@roman-khimov it should go to the testlib, easy to implement, will do

@roman-khimov roman-khimov transferred this issue from nspcc-dev/neofs-testcases Mar 12, 2024
@roman-khimov
Copy link
Member

OK if it doesn't interfere with tests (but I think we parse CLI output in some cases).

@evgeniiz321 evgeniiz321 transferred this issue from nspcc-dev/neofs-testlib Mar 28, 2024
@roman-khimov roman-khimov added the good first issue Good for newcomers label Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improving existing functionality good first issue Good for newcomers I3 Minimal impact S3 Minimally significant U2 Seriously planned
Projects
None yet
Development

No branches or pull requests

3 participants