Skip to content
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

chore: specify packageManager in package.json #2638

Merged
merged 1 commit into from
Apr 2, 2025

Conversation

wojtekmaj
Copy link
Contributor

Summary

This PR adds packageManager field to package.json, allowing users of Corepack (and perhaps other tools in the future) who already use a different version of Yarn to easily start contributing to the project.

Test Plan

  1. Enable Corepack by running corepack enable
    2a. Try installing this repo. Latest version of Yarn is going to be used; 4.7.0 at the moment of writing.
    2b. With this change applied, Corepack asks to install the latest Yarn Classic (1.22.22) and when the user agrees, it correctly installs the repository using Yarn Classic.

Checklist

  • Documentation is up to date.
  • Follows commit message convention described in CONTRIBUTING.md.
  • For functional changes, my test plan has linked these CLI changes into a local react-native checkout (instructions).

Copy link
Collaborator

@szymonrybczak szymonrybczak left a comment

Choose a reason for hiding this comment

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

thank you!

@szymonrybczak szymonrybczak merged commit 4ea2b35 into react-native-community:main Apr 2, 2025
4 of 8 checks passed
@wojtekmaj wojtekmaj deleted the wmaj/corepack branch April 2, 2025 15:56
@liamjones
Copy link
Contributor

liamjones commented Apr 3, 2025

Edit: wait, I'm an idiot - this is the CLI's own packageManager. 🤦 I was thinking of the template.

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.

3 participants