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

Analyzer plugin failure at the commandline should be displayed in the terminal #60175

Open
Tracked by #53402
srawlins opened this issue Feb 19, 2025 · 2 comments
Open
Tracked by #53402
Labels
analyzer-plugin area-devexp Developer Experience related issues (DevTools, IDEs, Analysis Server, completions, refactoring, etc) P2 A bug or feature request we're likely to work on type-enhancement A request for a change that isn't a bug

Comments

@srawlins
Copy link
Member

(I don't think they are right now)

These are all situations in which I think an error should be printed (to stdout or stderr, whatever we already do), and exit with a non-zero exit:

  • an exception that occurs while setting up the shared plugin entrypoint (like a pub get failure),
  • a malformed plugins section in analysis_options.yaml,
  • a plugin throws an exception while computing diagnostics,
@srawlins srawlins added analyzer-plugin area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. P2 A bug or feature request we're likely to work on type-enhancement A request for a change that isn't a bug labels Feb 19, 2025
@srawlins
Copy link
Member Author

CC @bwilkerson I think this should be pretty high up in the MVP priorities (sans feedback). I'm concerned that it is hard to evaluate the plugins story when the developer experience is poor. :/

@bwilkerson
Copy link
Member

I agree. I think there are two aspects to think about: how easy it is to write plugins and how easy it is to use a plugin. This falls into the latter bucket. Users need to be able to know whether the plugins they've enabled are behaving well or whether they're causing problems, both in the IDE and from the command-line.

@johnniwinther johnniwinther added area-devexp Developer Experience related issues (DevTools, IDEs, Analysis Server, completions, refactoring, etc) and removed area-devexp Developer Experience related issues (DevTools, IDEs, Analysis Server, completions, refactoring, etc) area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. labels Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analyzer-plugin area-devexp Developer Experience related issues (DevTools, IDEs, Analysis Server, completions, refactoring, etc) P2 A bug or feature request we're likely to work on type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

3 participants