Skip to content

Assists for linter rules #59868

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

Open
FMorschel opened this issue Jan 8, 2025 · 0 comments
Open

Assists for linter rules #59868

FMorschel opened this issue Jan 8, 2025 · 0 comments
Labels
area-devexp For issues related to the analysis server, IDE support, linter, `dart fix`, and diagnostic messages. devexp-quick-fix Issues with analysis server (quick) fixes P4

Comments

@FMorschel
Copy link
Contributor

FMorschel commented Jan 8, 2025

After some discussion over at #55985, we got to a decision to eventually move the fixes related to pubspec.yaml over to the server. I'm guessing the work for analysis_options.yaml would be similar if not the same. I'd like to request an assist for lint rules when we reach that point.

It would be good if we had an assist for the following (or cases where rules are incompatible/opposite) to swap between rules.

linter:
  rules:
    - prefer_single_quotes

Would show something like Swap this for 'prefer_double_quotes' or similar. I think this would be a good way of teaching people about opposite rules which they might not be familiar with and it would help people like me with a memory that doesn't seem to want to remember the rules' names and always need to look them up (sometimes I only remember the opposite rule and for setting up a new project this would be great).

This would go nicely with #57034.

@FMorschel FMorschel added the legacy-area-analyzer Use area-devexp instead. label Jan 8, 2025
@bwilkerson bwilkerson added devexp-quick-fix Issues with analysis server (quick) fixes P4 labels Jan 21, 2025
@bwilkerson bwilkerson added area-devexp For issues related to the analysis server, IDE support, linter, `dart fix`, and diagnostic messages. and removed legacy-area-analyzer Use area-devexp instead. labels Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-devexp For issues related to the analysis server, IDE support, linter, `dart fix`, and diagnostic messages. devexp-quick-fix Issues with analysis server (quick) fixes P4
Projects
None yet
Development

No branches or pull requests

2 participants