Skip to content

doc: clarify Corepack removal in v25+ #57813

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 4 commits into from
Closed

Conversation

trivikr
Copy link
Member

@trivikr trivikr commented Apr 10, 2025

Alternative to #57663

This preserves the existing documentation page for corepack which has backlinks from all over the internet.
It can be removed close to April 2028, after corepack is completely removed from all Node.js versions.

Refs:

@trivikr trivikr added dont-land-on-v18.x dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. dont-land-on-v24.x PRs that should not land on the v18.x-staging branch and should not be released in v24.x. labels Apr 10, 2025
@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Apr 10, 2025
@trivikr trivikr requested a review from aduh95 April 10, 2025 04:53
Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

LGTM with the grammar suggestion

@trivikr trivikr requested a review from ljharb April 10, 2025 06:19
@aduh95
Copy link
Contributor

aduh95 commented Apr 10, 2025

The commit message do not match our guidelines, the first word after the subsystem should be an imperative verb.

trivikr and others added 3 commits April 10, 2025 15:53
Co-authored-by: Mike McCready <66998419+MikeMcC399@users.noreply.github.com>
@trivikr trivikr force-pushed the trivikr/doc-corepack branch from f163dc7 to 3b95127 Compare April 10, 2025 15:54
@trivikr trivikr changed the title doc: corepack no longer distributed in v25+ doc: clarify Corepack removal in v25+ Apr 10, 2025
@trivikr
Copy link
Member Author

trivikr commented Apr 10, 2025

@aduh95 I've updated the first commit to use imperative verb after subsystem: doc: clarify Corepack removal in v25+

@trivikr
Copy link
Member Author

trivikr commented Apr 10, 2025

Closing this PR, as the existing documentation can be edited better with highlight and call to action just like it's done for other modules for all Node.js versions. That PR is posted in #57825

@trivikr trivikr closed this Apr 10, 2025
@trivikr trivikr deleted the trivikr/doc-corepack branch April 10, 2025 17:52
@trivikr
Copy link
Member Author

trivikr commented Apr 10, 2025

I'll post a new PR specific to v25.x after #57825 is merged.
There we can change the language to mark corepack as deprecated, just like we did in punycode.

@trivikr
Copy link
Member Author

trivikr commented Apr 14, 2025

We're instead trying to enable redirect from corepack latest documentation in nodejs/nodejs.org#7643 (review)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. dont-land-on-v20.x PRs that should not land on the v20.x-staging branch and should not be released in v20.x. dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. dont-land-on-v24.x PRs that should not land on the v18.x-staging branch and should not be released in v24.x.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants