+
Skip to content
This repository was archived by the owner on Aug 31, 2023. It is now read-only.

Fix pnpm install command #3688

Merged
merged 1 commit into from
Nov 12, 2022
Merged

Fix pnpm install command #3688

merged 1 commit into from
Nov 12, 2022

Conversation

levrik
Copy link
Contributor

@levrik levrik commented Nov 12, 2022

According to https://pnpm.io/cli/install pnpm install is used to install a whole project and not to add a single dependency. pnpm add should be used instead.
I just tried to use pnpm install like shown here and it surprisingly worked but since this is not documented it might be better to use pnpm add in Rome's documentation.

According to https://pnpm.io/cli/install `pnpm install` is used to install a whole project and not to add a single dependency. `pnpm add` should be used instead.
@levrik levrik requested a review from a team November 12, 2022 16:49
@netlify
Copy link

netlify bot commented Nov 12, 2022

Deploy Preview for docs-rometools ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit c21e3ff
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/636fcea0cefa8d000abbe899
😎 Deploy Preview https://deploy-preview-3688--docs-rometools.netlify.app/guides/getting-started
📱 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 site settings.

@sebmck
Copy link
Contributor

sebmck commented Nov 12, 2022

Thank you!

@sebmck sebmck merged commit 0504865 into rome:main Nov 12, 2022
@levrik levrik deleted the patch-1 branch November 12, 2022 19:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载