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

Warnings should be optional (or removed entirely) #31

Closed
ProLoser opened this issue Oct 4, 2016 · 1 comment
Closed

Warnings should be optional (or removed entirely) #31

ProLoser opened this issue Oct 4, 2016 · 1 comment

Comments

@ProLoser
Copy link

ProLoser commented Oct 4, 2016

They take up precious real estate, and some of these warnings are extremely opinionated and I may choose to never address them. I may have a really long function and I can't or won't ever do anything about it, so I should not be forced to lose visual real estate. You've provided no way to toggle this stuff off or adjust it. And frankly, this doesn't even seem like the responsibility of YOUR package, it's just causing bloat and complexity.

If you REALLY want to provide intuitive linting in symbol view (which I don't really think is necessary since I can see linting errors in my scrollbar, gutter, and minimap) then you should instead simply count how many linter errors and warnings are in each function, etc, and simply give me a warning or error icon on the symbol that contains such issues with a count of how many errors or warnings are found.

However, I ultimately feel this entire feature just needs to be removed. Completely superfluous and not worth the time to work on since it's done in such a limited manner.

@hex13
Copy link
Owner

hex13 commented Sep 20, 2017

Now warnings are optional and can be disabled/enabled in settings (thanks to @srguiwiz )

@hex13 hex13 closed this as completed Sep 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants