search

nodejs / corepack

Zero-runtime-dependency package acting as bridge between Node projects and their package managers
MIT License
2.31k stars 146 forks source link

Adds a note clarifying how to add new package managers #408

Closed arcanis closed 4 months ago

arcanis commented 4 months ago

We didn't document the process to add new package managers to Corepack. I added a rough draft based on the discussions we had around the topic lately, feel free to suggest improvements in wording or content.

GeoffreyBooth commented 4 months ago

@arcanis I think that PRs like this that involve governance (also design decisions and the like) need much more review than being opened and merged in a matter of hours. The TSC should be tagged, as well as relevant teams like @nodejs/npm and @nodejs/package-maintenance, and we should follow the standard rules for landing PRs on the main repo where they stay open for a week if they have one approval, or 48 hours with two approvals.

Alternatively we could define Corepack’s governance and design in the core repo.