-
Notifications
You must be signed in to change notification settings - Fork 223
Introduce an analyzer plugin for the test package. #2461
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
base: master
Are you sure you want to change the base?
Conversation
PR HealthChangelog Entry ✔️
Changes to files need to be accounted for in their respective changelogs. This check can be disabled by tagging the PR with |
| @@ -0,0 +1,13 @@ | |||
| # test_analyzer_plugin | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably add something to the package:test readme as well, informing users of this plugin and how to enable it for their project.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we are very "soft launching" analyzer plugins ATM, I think we don't want to advertise it there yet. WDYT?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Up to you I guess, but we won't get usage if we don't advertise it
|
I've not yet read the whole changes, but what about an annotation on I'd love to contribute to it if this is something you are interested. |
That's a fun idea, you could open an issue against this repo. For first-party matchers in the matcher package, I don't think it would be worth the extra complexity. So the feature would be for custom matchers implemented outside of the matcher and test packages. |
e6bd69d to
2635b72
Compare
|
OK I'd love to land this actually. Esp with Dart 3.10 out today. Anyone know what the failing Publish check is about? I can't make heads or tails of it. |
Note: I'm not strongly suggesting this PR be accepted today as is. But I'm offering it up.
As part of the new analyzer plugin system, I'd like to offer some first-party-maintained analyzer plugins. I don't intend for us to maintain complex plugins necessarily. But good bang-for-our-buck plugins have some great benefits:
If there are other static analyses that the team has dreamed of, I'd love to evaluate them for complexity and possibly add more static analyses to this plugin.
Again, we don't need to land this any time soon, if the team is hesitant to check this in, maintain it, etc. The new analyzer system is not officially released, and the API is probably not super stable. I'm opening this PR for the discussion, and to have URL to point to as a real world analyzer plugin example.