Skip to content

Comments

Inline license validation code#172

Merged
wraithgar merged 3 commits intomainfrom
gar/validate-npm-package-license
Feb 18, 2026
Merged

Inline license validation code#172
wraithgar merged 3 commits intomainfrom
gar/validate-npm-package-license

Conversation

@wraithgar
Copy link
Member

This is three commits:

  • deps: remove validate-npm-package-license
  • deps: spdx-expression-parse@4.0.0
  • fix: inline license validation code

It removes validate-npm-package-license which has been unmaintained for some time and is out of date with its dependencies.
It inlines the 'validForNewPackage' flag calculation from that package, which is all we were using it for.
It also adds the spdx-expression-parse package that was being used to parse the license field, at its latest version (no longer outdated).

@wraithgar wraithgar requested a review from a team as a code owner February 18, 2026 20:41
@wraithgar wraithgar merged commit 09b981d into main Feb 18, 2026
17 checks passed
@wraithgar wraithgar deleted the gar/validate-npm-package-license branch February 18, 2026 21:21
@github-actions github-actions bot mentioned this pull request Feb 18, 2026
wraithgar pushed a commit that referenced this pull request Feb 19, 2026
🤖 I have created a release *beep* *boop*
---


## [7.0.5](v7.0.4...v7.0.5)
(2026-02-18)
### Bug Fixes
*
[`09b981d`](09b981d)
[#172](#172) inline license
validation code (@wraithgar)
### Dependencies
*
[`cb3e677`](cb3e677)
[#172](#172)
`spdx-expression-parse@4.0.0`
*
[`61738b2`](61738b2)
[#172](#172) remove
validate-npm-package-license
### Chores
*
[`bd3b7ea`](bd3b7ea)
[#169](#169) bump
@npmcli/template-oss from 4.28.0 to 4.28.1 (#169) (@dependabot[bot],
@npm-cli-bot)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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