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
The current implementation of "disable rule for this line" results in unintentional behavior when the action is applied to a JSX element or attribute:
disable-rule-screencap.mov
The inline disable syntax is much more appropriate for this scenario:
It would be really nice to expose a third quick-fix action to "disable rule for this line (inline)" for use in this scenario.
Alternatively it could be a user setting whether the existing action uses eslint-disable-line or eslint-disable-next-line. But I think surfacing a third option is better, because the choice is context-sensitive.