Skip to content

feat: NX-15190 switch to centralized npm license check#44

Merged
Hinne1 merged 9 commits intomainfrom
feature/NX-15190-centralized-node-license-check
Mar 3, 2026
Merged

feat: NX-15190 switch to centralized npm license check#44
Hinne1 merged 9 commits intomainfrom
feature/NX-15190-centralized-node-license-check

Conversation

@smini-digital
Copy link
Contributor

@smini-digital smini-digital commented Feb 25, 2026

Summary

  • Replace unmaintained license-checker-rseidelsohn with the centralized npm-license-validator composite action
  • Removes the need to install all Node.js dependencies before license checking
  • License data is now read directly from package-lock.json via CycloneDX SBOM

References

@gemini-code-assist
Copy link

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@smini-digital smini-digital changed the title feat: NX-15190 use centralized node license check feat: NX-15190 switch to centralized npm license check Feb 26, 2026
@smini-digital smini-digital self-assigned this Feb 26, 2026
Copy link
Contributor

@Hinne1 Hinne1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM — consistent with the centralized license check pattern across all repos. CI green.

@Hinne1 Hinne1 merged commit 578e665 into main Mar 3, 2026
3 checks passed
@Hinne1 Hinne1 deleted the feature/NX-15190-centralized-node-license-check branch March 3, 2026 10:31
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.

2 participants