Skip to content

Nx enforce-module-boundaries doesn't support call expressions and variable declarations#406

Open
qing-lin-007 wants to merge 1 commit intonrwl:masterfrom
qing-lin-007:enforce-module-boundaries
Open

Nx enforce-module-boundaries doesn't support call expressions and variable declarations#406
qing-lin-007 wants to merge 1 commit intonrwl:masterfrom
qing-lin-007:enforce-module-boundaries

Conversation

@qing-lin-007
Copy link

@qing-lin-007 qing-lin-007 commented Jan 14, 2026

What Changed

  1. created a new lib via: npx nx g @nx/js:lib libs/shared/mylib
  2. turn off the rule for the new lib: "@typescript-eslint/no-var-requires": "off"
  3. in the new lib created a file that should throw linting errors but it doesn't: libs/shared/mylib/src/lib/mylib.ts

@netlify
Copy link

netlify bot commented Jan 14, 2026

Deploy Preview for nrwl-nx-examples-dep-graph ready!

Name Link
🔨 Latest commit 0e6ab93
🔍 Latest deploy log https://app.netlify.com/projects/nrwl-nx-examples-dep-graph/deploys/696756debe63930008606b80
😎 Deploy Preview https://deploy-preview-406--nrwl-nx-examples-dep-graph.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@nx-cloud
Copy link

nx-cloud bot commented Jan 14, 2026

View your CI Pipeline Execution ↗ for commit 0e6ab93

Command Status Duration Result
nx affected --base=144ce90d72ca14e0fb43d39be94a... ✅ Succeeded <1s View ↗
nx affected --base=144ce90d72ca14e0fb43d39be94a... ✅ Succeeded 14s View ↗
nx-cloud record -- nx format:check --base=144ce... ✅ Succeeded 3s View ↗

☁️ Nx Cloud last updated this comment at 2026-01-14 08:44:03 UTC

@qing-lin-007 qing-lin-007 changed the title Nx enforce-module-boundaries should support call expressions and variable declarations Nx enforce-module-boundaries doesn't support call expressions and variable declarations Jan 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant