There are several popular packages that currently have to be ignored when running dependency_validator because they are only used for the builder they provide, which is something that gets applied automatically when running build_runner and not something we can detect via imports in the source code.
Changes
Look up the source for each dependency in ~/.pub-cache to see if a build.yaml exists, parse it, and see if any builders are defined. If a dependency defines builder(s), we should consider it "used".
Release Notes
Review
See CONTRIBUTING.md for more details on review types (+1 / QA +1 / +10) and code review process.
Please review:
QA Checklist
[ ] Tests were updated and provide good coverage of the changeset and other affected code
Motivation
There are several popular packages that currently have to be ignored when running dependency_validator because they are only used for the builder they provide, which is something that gets applied automatically when running build_runner and not something we can detect via imports in the source code.
Changes
Look up the source for each dependency in ~/.pub-cache to see if a build.yaml exists, parse it, and see if any builders are defined. If a dependency defines builder(s), we should consider it "used".
Release Notes
Review
See CONTRIBUTING.md for more details on review types (+1 / QA +1 / +10) and code review process.
Please review:
QA Checklist
Merge Checklist
While we perform many automated checks before auto-merging, some manual checks are needed: