Skip to content

Update package manager instructions in README.md#16427

Merged
wackerow merged 2 commits into
ethereum:devfrom
khalil7044:patch-2
Oct 6, 2025
Merged

Update package manager instructions in README.md#16427
wackerow merged 2 commits into
ethereum:devfrom
khalil7044:patch-2

Conversation

@khalil7044
Copy link
Copy Markdown
Contributor

Added a short note at the end of README.md for Ubuntu and Debian users who may need to install nodejs and npm before using yarn install

Description

Related Issue

Added a short note at the end of README.md for Ubuntu and Debian users who may need to install nodejs and npm before using yarn install
@github-actions github-actions Bot added the documentation 📖 Change or add documentation label Oct 2, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 2, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit e833d85
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/68e3f55a0e3e5800086f02ca
😎 Deploy Preview https://deploy-preview-16427--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

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

Thanks @khalil7044... This project uses pnpm as it's primary package manager at this point and using yarn could lead to package inconsistencies, and add undesired changes if users take this approach before submitting PRs. Ideally we make this clear instead of adding more instructions about how to use yarn.

(Would also try to find a better spot for such an update than below the "Back to top" link 😉)

@wackerow wackerow changed the title Update README.md Update package manager instructions in README.md Oct 2, 2025
Copy link
Copy Markdown
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

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

I've gone ahead and adjusted this to remove yarn comments, replacing with corepack enable and pnpm install, and moved this up in the section that handles these steps. Thanks @khalil7044!

@wackerow wackerow merged commit 96a6898 into ethereum:dev Oct 6, 2025
5 of 6 checks passed
@gitpoap-bot
Copy link
Copy Markdown

gitpoap-bot Bot commented Oct 6, 2025

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2025 Ethereum.org Contributor:

GitPOAP: 2025 Ethereum.org Contributor GitPOAP Badge

Join the [ethereum.org Discord server](https://ethereum.org/discord) to explore more ways to contribute to the project. Depending on the tasks you complete, you may also unlock additional rewards. Visit [ethereum.org/contributing](https://ethereum.org/contributing) to learn more.

Head to gitpoap.io & connect your GitHub account to mint!Keep buidling, keep learning, and let's grow the Ethereum open-source community together 🌱

Learn more about GitPOAPs here.

@wackerow
Copy link
Copy Markdown
Member

wackerow commented Oct 6, 2025

@all-contributors please add @khalil7044 for infra

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation 📖 Change or add documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants