nx
nx copied to clipboard
docs(linter): [eslint-plugin] improve error message for forbidden imp…
…orts of lazy-loaded libs
- Specify that static imports are forbidden
- Include target library name and source file path containing a dynamic import that triggered this rule
Related Issue(s)
Related to #17425
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| nx-dev | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jun 19, 2023 2:38pm |
☁️ Nx Cloud Report
CI is running/has finished running commands for commit f800d35a5a22a12cc1bd67bef2a10eeda1da54b1. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.
📂 See all runs for this branch
✅ Successfully ran 8 targets
nx affected -t e2e-macos --parallel=1 --base=1fda800b332af491e14f2d5d213d63af952405aa --head=f800d35a5a22a12cc1bd67bef2a10eeda1da54b1nx affected --target=e2e --base=1fda800b332af491e14f2d5d213d63af952405aa --head=f800d35a5a22a12cc1bd67bef2a10eeda1da54b1 --parallel=1nx affected --target=test --base=1fda800b332af491e14f2d5d213d63af952405aa --head=f800d35a5a22a12cc1bd67bef2a10eeda1da54b1 --parallel=1nx affected --target=build --base=1fda800b332af491e14f2d5d213d63af952405aa --head=f800d35a5a22a12cc1bd67bef2a10eeda1da54b1 --parallel=3nx run-many -t check-imports check-commit check-lock-files depcheck check-codeowners documentation --parallel=1 --no-dtenx affected --target=lint --base=1fda800b332af491e14f2d5d213d63af952405aa --head=f800d35a5a22a12cc1bd67bef2a10eeda1da54b1 --parallel=3nx-cloud record -- nx format:check --base=1fda800b332af491e14f2d5d213d63af952405aa --head=f800d35a5a22a12cc1bd67bef2a10eeda1da54b1nx documentation --no-dte
Sent with 💌 from NxCloud.
Thank you @miluoshi,
I left one small improvement point.
(also PR is failing due to format:check, please run pnpm nx format and push the re-formatted code)
Thanks for the suggestion. Updated.
Thank you, I will add one unit test to test this and then we can merge it.
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.