Skip to content

Conversation

@owlstronaut
Copy link
Contributor

@owlstronaut owlstronaut commented Jul 21, 2025

This pull request updates terminology for clarity and inclusivity, modifies validation logic, and enhances test coverage for the validate-npm-package-name module. The most important changes include renaming to exclusion list, updating error messages, and adding new test cases for scoped package validation.

@owlstronaut owlstronaut requested a review from a team as a code owner July 21, 2025 16:40
@wraithgar
Copy link
Member

We'll have to dig into this on the registry side but we may just have to allow these. I know builtins especially should be allowed cause that's the whole point of scoped packages. @gar/http is very much an allowed package name.

@owlstronaut owlstronaut force-pushed the scoped-name-validation branch from 9006ddf to fee6521 Compare July 21, 2025 19:54
@owlstronaut owlstronaut force-pushed the scoped-name-validation branch from fee6521 to f072c2a Compare July 21, 2025 20:06
@wraithgar wraithgar changed the title fix: scoped names validate blacklist/underscore/core module fix: scoped names validate exclusionlist/underscore/core module Jul 21, 2025
@owlstronaut owlstronaut merged commit 186f541 into main Jul 21, 2025
23 checks passed
@owlstronaut owlstronaut deleted the scoped-name-validation branch July 21, 2025 20:11
@github-actions github-actions bot mentioned this pull request Jul 21, 2025
owlstronaut pushed a commit that referenced this pull request Jul 21, 2025
🤖 I have created a release *beep* *boop*
---


##
[6.0.2](v6.0.1...v6.0.2)
(2025-07-21)
### Bug Fixes
*
[`186f541`](186f541)
[#142](#142) scoped
names validate exclusionlist/underscore/core module (#142)
(@owlstronaut)
### Chores
*
[`8123f73`](8123f73)
[#140](#140) bump
@npmcli/template-oss from 4.24.4 to 4.25.0 (#140) (@dependabot[bot],
@owlstronaut)

---
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