Skip to content

"ignore [lint]" quick-fix doesn't correctly add to existing ignores #3407

Closed

Description

@DanTup
void main(List<String> args) {
  // ignore: foo
  var a = 1;
}

Using the fix to ignore unused_local_variable just adds an additional line (making ignore: foo not work), but it should merge them (// ignore: foo, unused_local_variable).

Activity

  1. added this to the v3.24.0 milestone on Jun 15, 2021
  2. DanTup commented on Jun 17, 2021

    @DanTup
    MemberAuthor

    The SDK versions of these ignore fixes handles this properly:

    dart-lang/sdk@2e411e5

    It'll need an SDK update to get, at which point VS Code's fixes will be hidden.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    in editorRelates to code editing or language featuresis bug

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions