Skip to content

Update messages.yamls language version overrides #60093

Open
@FMorschel

Description

@FMorschel

We have some different messages.yaml files. These are the ones I could find/remember:

  • pkg/analyzer/messages.yaml
  • pkg/front_end/messages.yaml
  • pkg/liner/messages.yaml
  • pkg/analyzer/messages.yaml

This is coming from a comment on https://dart-review.googlesource.com/c/sdk/+/407080 from @bwilkerson:

This language marker doesn't appear to be necessary (pasting the example without it into an editor produces the expected diagnostic).

We should probably just get rid of all %language markers, replacing them with ``// dart=` comments where they're actually required. (But not as part of this CL.)

About places like these:

%language=2.18

That should become something like:

// @dart = 2.8

Because of INVALID_LANGUAGE_VERSION_OVERRIDE (if I'm not mistaken).

Metadata

Metadata

Assignees

No one assigned

    Labels

    P3A lower priority bug or feature requestanalyzer-technical-debtarea-devexpFor issues related to the analysis server, IDE support, linter, `dart fix`, and diagnostic messages.type-taskA well-defined stand-alone task

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions