You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When I start working on a new code base one of the first things I tend to do is add the effective dart lints.
This typically yields thousands of errors.
A moderate no. of the lints are not valid.
It would be really convenient if the dart-code offered an 'add ignore' option quick fix for these lints.
On Tue, 26 May 2020 at 01:09, Danny Tuppeny ***@***.***> wrote:
This is supported, but you have to enable it with the following setting:
"dart.showIgnoreQuickFixes": true,
That should add additional quick-fix entries to ignore. Let me know if you
have any issues with it!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#2487 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAG32OGHT6OGK3BDRI5ZVK3RTKCZFANCNFSM4NIFW45Q>
.
The implementation of this is slightly hacky - it's done client-side and not in the language server, so it works by looking at errors in the current file and fabricates the assist, then just inserts a comment in the line above. It's possible for some code it might insert incorrectly.
I tend to ship things like that off-by-default to get a little feedback usually from the person that originally asked for it, or my own use). There hasn't been much, but I've used it myself enough that I'm reasonable sure of it, so I'll look at switching it on by default in an upcoming version.
Activity
DanTup commentedon May 25, 2020
This is supported, but you have to enable it with the following setting:
That should add additional quick-fix entries to ignore. Let me know if you have any issues with it!
bsutton commentedon May 25, 2020
DanTup commentedon May 26, 2020
The implementation of this is slightly hacky - it's done client-side and not in the language server, so it works by looking at errors in the current file and fabricates the assist, then just inserts a comment in the line above. It's possible for some code it might insert incorrectly.
I tend to ship things like that off-by-default to get a little feedback usually from the person that originally asked for it, or my own use). There hasn't been much, but I've used it myself enough that I'm reasonable sure of it, so I'll look at switching it on by default in an upcoming version.
[-]Offer option to 'ignore' lint[/-][+]Enable assist to ignore lints by default[/+]bsutton commentedon May 26, 2020
I've used this a couple of times now and it seems to be working fine.
I would be inclined to ship it on by default.
Worst case it inserts it incorrectly and the user just has to ctrl-z to fix the issue.
Enable ignore lint quick fix