Skip to content

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

Merged
leosvelperez merged 2 commits intomasterfrom
eslint-plugin/fix-ng-packagr-secondary-entry-point
May 28, 2025
Merged

fix(linter): handle ng-package.json file with no lib.entryFile in @nx/enforce-module-boundaries rule#31360
leosvelperez merged 2 commits intomasterfrom
eslint-plugin/fix-ng-packagr-secondary-entry-point

Conversation

@leosvelperez
Copy link
Copy Markdown
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
@vercel
Copy link
Copy Markdown

vercel Bot commented May 27, 2025

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

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview May 28, 2025 7:19am

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented May 27, 2025

View your CI Pipeline Execution ↗ for commit 95b9e22.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 6m 18s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 19s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 6s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 4s View ↗
nx documentation ✅ Succeeded 1m 29s View ↗

☁️ Nx Cloud last updated this comment at 2025-05-28 07:21:02 UTC

@leosvelperez leosvelperez merged commit c7a9c71 into master May 28, 2025
6 checks passed
@leosvelperez leosvelperez deleted the eslint-plugin/fix-ng-packagr-secondary-entry-point branch May 28, 2025 07:45
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2025

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.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants