Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change vague message for sort_constructors_first #4556

Closed
Levi-Lesches opened this issue Jul 12, 2023 · 1 comment · Fixed by #4570
Closed

Change vague message for sort_constructors_first #4556

Levi-Lesches opened this issue Jul 12, 2023 · 1 comment · Fixed by #4570
Labels
P3 A lower priority bug or feature request type-documentation A request to add or improve documentation

Comments

@Levi-Lesches
Copy link

Levi-Lesches commented Jul 12, 2023

Describe the issue
Currently, the lint message is Invalid location for a constructor declaration. That doesn't exactly tell you what to change or how to fix the problem.

To Reproduce

class A {
  final int b;
  A(this.b);  // INFO | LINT Invalid location for a constructor declaration.
}

Expected behavior
Something clearer, like what's currently on the page for the lint:

Sort constructor declarations before other members.

Additional context
https://dart.dev/tools/linter-rules/sort_constructors_first

@pq
Copy link
Member

pq commented Jul 17, 2023

/fyi @bwilkerson

@pq pq added type-documentation A request to add or improve documentation ux P3 A lower priority bug or feature request labels Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P3 A lower priority bug or feature request type-documentation A request to add or improve documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants