Confusion around how --quiet and --fix should behave together
See original GitHub issueSee #8353 for background.
I think the best way to fix this is to allow users the option to go either way and to be very explicit about how the options work.
Here’s one proposal along those lines:
- Rename --quiet to --no-report-warnings (supporting --quiet as an alias and preserving its behavior), to avoid reporting warnings but possibly fix them
- Add option --no-fix-warnings to support not running autofix on warnings but possibly report them (no effect if
--fix
not specified) - Users can use both to completely ignore warnings altogether
With this proposal, our CLI options become more clear and more flexible with regard to whether warnings are reported/logged (--quiet
/--no-report-warnings
) and whether warnings are fixed (--no-fix-warnings
).
I’ll champion.
Issue Analytics
- State:
- Created 6 years ago
- Reactions:2
- Comments:14 (14 by maintainers)
Top Results From Across the Web
Stonewalling: The Silent Relationship Killer - Banner Health
Stonewalling is a refusal to communicate or cooperate, and this defense mechanism could be a relationship killer. Read on to learn the signs ......
Read more >What Couples Should Know About the Silent Treatment
Silence you when you attempt to assert yourself by refusing to talk. Communicate disdain or contempt in order to maintain the silence.
Read more >How to Find Clarity When You're Confused About What to Do
One must remember that in order to solve a problem, one must first rise above it (that comes from a quiet mind that...
Read more >Confusion - symptoms, causes and prevention - Healthdirect
Try to have a calm and peaceful environment where the person is not constantly distracted by noise or too much activity going on...
Read more >20 Ways to Tell if a Men Is Confused About His Feelings for You
Reassure him It's better if he comes up with his own decision. While he's taking his time, keep your cool and make sure...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Just arranged. Granted, looks good to me if we change that ※ as a bug.
--quiet
--fix
--fix --quiet
@mysticatea Yes, I think you’re right, that behavior is unexpected (and was the root of the original issue). I like the direction you’re going, but maybe we could bikeshed the name a little bit?
How about this?
--warnings=report-and-fix
(default)--warnings=report-only
(no fix, but report warnings)--warnings=none
(don’t report or fix warnings, alias--quiet
)