Skip to content

no_entrypoint_imports #86

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

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sourcegraph-wk
Copy link

Problem

An entrypoint import can be defined as an import within lib/src that imports a file within lib/*.dart (the entrypoints of a dart package). These imports are always unnecessary, and can contribute to slower dart build performance.

Solution

This PR naively removes every entrypoint import within the repo, and updates the gha-dart/.../checks.yaml to the latest version. In order to merge this PR a few manual changes must be performed:

  • Navigate to every file which has analysis errors caused from the missing import. Rely on intellisense to import the necessary symbol from the specific file, not the entrypoint import
  • Implement the no_entrypoint_imports additional-checks option in [email protected] that was added in this PR. This will prevent new entrypoint imports from getting added once this PR merges

Our request to teams is to perform the above tasks, so we can default enable the no_entrypoint_imports check for all gha-dart consumers. Please reach out to #support-frontend-dx for any questions or issues

QA

  • This pr is heavily dependant on the analysis server, and as such CI passes should be sufficient

Created by Sourcegraph batch change Workiva/no_entrypoint_imports.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant