quick-lint-js icon indicating copy to clipboard operation
quick-lint-js copied to clipboard

12$: Improve reporting of wrongly ordered or duplicated class member modifiers

Open strager opened this issue 2 years ago • 1 comments

class C {
  async static f() {}  // error
  static public g() {}  // no error, but should
  async async async async h() {}  // no error, but should
  readonly static x;  // (TypeScript) error
}

parse_and_visit_class_or_interface_member now has bump_vector<modifier, ...> which we can easily analyze to detect misordered and duplicate modifiers. Our current analysis is incomplete.

strager avatar Jun 03 '22 23:06 strager

We should ideally merge E0269 and E0232.

strager avatar Sep 09 '22 06:09 strager

abstract is a modifier we don't have any ordering diagnostic for.

strager avatar Sep 23 '22 03:09 strager