Skip to content

fix(linter): handle ng-package.json file with no lib.entryFile in @nx/enforce-module-boundaries rule #31360

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

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

Conversation

leosvelperez
Copy link
Member

Current Behavior

When an ng-package.json file of an Angular library secondary entry point does not specify lib.entryFile, the @nx/enforce-module-boundaries rule throws an error. The ng-package.json file of an Angular secondary entry point can be as simple as {}, but it would cause the rule to throw an error.

Expected Behavior

The @nx/enforce-module-boundaries rule should correctly handle an ng-package.json file of an Angular library secondary entry point that does not specify lib.entryFile. The property should default to src/public_api.ts.

@leosvelperez leosvelperez self-assigned this May 27, 2025
@leosvelperez leosvelperez requested a review from a team as a code owner May 27, 2025 15:39
@leosvelperez leosvelperez requested a review from meeroslav May 27, 2025 15:39
Copy link

vercel bot commented May 27, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) May 27, 2025 3:40pm

Copy link

nx-cloud bot commented May 27, 2025

View your CI Pipeline Execution ↗ for commit 294799d.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ❌ Failed 33m 31s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 17s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 3s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 2s View ↗
nx documentation ✅ Succeeded 1m 13s View ↗

☁️ Nx Cloud last updated this comment at 2025-05-27 16:20:00 UTC

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