Skip to content
This repository has been archived by the owner on Mar 23, 2024. It is now read-only.

Verbose should provide more information #1522

Closed
hzoo opened this issue Jul 10, 2015 · 11 comments
Closed

Verbose should provide more information #1522

hzoo opened this issue Jul 10, 2015 · 11 comments

Comments

@hzoo
Copy link
Member

hzoo commented Jul 10, 2015

(2.0) We will need to change the docs for this since it's already going to be on right? And the options will have to change to turn it off instead of on?

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@markelog
Copy link
Member

Did we discuss this?

@markelog markelog added the cli label Jul 10, 2015
@hzoo
Copy link
Member Author

hzoo commented Jul 10, 2015

I have to find it but I remember talking about it

@markelog
Copy link
Member

I'd say you need to know rule name only if want to disable it which seems as a rare case, otherwise you just need to fix the error.

Plus no other linter does it.

@hzoo
Copy link
Member Author

hzoo commented Jul 10, 2015

Oh it was a comment from @mikesherov so don't know if we did then #1241 (comment)

@mikesherov
Copy link
Contributor

Yes, I thought we should do verbose by default because new users when configuring need this info.

@hzoo
Copy link
Member Author

hzoo commented Aug 7, 2015

Oh and the other reason is when reporting a bug for a rule

@hzoo
Copy link
Member Author

hzoo commented Oct 1, 2015

Let's do this

@qfox
Copy link
Member

qfox commented Oct 1, 2015

👍

@markelog
Copy link
Member

markelog commented Oct 1, 2015

Well, that would practically eliminate this flag and it would be major change, so in any case it would have to wait for 3.0.

To prove these hypothesises we would need to fix #1064, since we didn't received actual opinion from users.

@hzoo hzoo added this to the 3.0 milestone Oct 2, 2015
@markelog
Copy link
Member

markelog commented Oct 6, 2015

Hm, after some thought, i think we should do that, even without #1064, what we want to do with --verbose flag though? remove it?

@hzoo
Copy link
Member Author

hzoo commented Oct 7, 2015

We could make it not do anything? Is there other information we could be telling the user?

@mdevils mdevils mentioned this issue Oct 8, 2015
16 tasks
@hzoo hzoo changed the title Misc: turn on verbose by default Verbose should provide more information Jan 30, 2016
@hzoo hzoo closed this as completed in dc46843 Feb 28, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants