Skip to content

docs: update description for Yarn updates#2408

Closed
MikeMcC399 wants to merge 1 commit intonodejs:mainfrom
MikeMcC399:yarn-update-description
Closed

docs: update description for Yarn updates#2408
MikeMcC399 wants to merge 1 commit intonodejs:mainfrom
MikeMcC399:yarn-update-description

Conversation

@MikeMcC399
Copy link
Contributor

Description

Reformulate the description for Yarn updates in the CONTRIBUTING document.

Motivation and Context

  • The CONTRIBUTING document describes updating Yarn v1 Classic versions. The details are no longer needed as Yarn v1 is frozen at 1.22.22 and hasn't had an update since 2024.
  • Yarn v1 is planned for removal with Node.js 26
  • The example ./update.sh 6.10 is incorrect, as the script only accepts Node.js major versions as a parameter. See update.sh. Possibly 6,10 was intended, as a list of major versions.

Testing Details

N/A - documentation only

Types of changes

  • Documentation
  • Version change (Update, remove or add more Node.js versions)
  • Variant change (Update, remove or add more variants, or versions of variants)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Other (none of the above)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING.md document.
  • All new and existing tests passed.

@MikeMcC399
Copy link
Contributor Author

@nschonni
Copy link
Member

I opened #2416 that might conflict with some of these explainations. Open to suggested wording on that one if it lands.

@MikeMcC399 MikeMcC399 marked this pull request as draft March 11, 2026 06:12
@MikeMcC399
Copy link
Contributor Author

@nschonni

I opened #2416 that might conflict with some of these explainations. Open to suggested wording on that one if it lands.

I like your proposed change, so I've changed this one to draft for the time being. I'll expect to ultimately close it unmerged.

@MikeMcC399
Copy link
Contributor Author

I've reviewed and tested #2416, so I'm now closing this one.

@MikeMcC399 MikeMcC399 closed this Mar 11, 2026
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