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

Add support for pnpm #8

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

sishenhei7
Copy link

@sishenhei7 sishenhei7 commented Dec 1, 2021

Pnpm uses copies of dependency and dependencies are stored in .pnpm folder.
When using pnpm, yorkie cannot correctly detect the depDir.
I fixed it in this pr.

@EthanWan
Copy link

EthanWan commented Dec 6, 2021

mark

5 similar comments
@xinlei3166
Copy link

mark

@arika163
Copy link

mark

@chengpeiquan
Copy link

mark

@wangxdmm
Copy link

mark

@Ge-yuan-jun
Copy link

mark

@littlekie
Copy link

The latest code for this NPM package does not fix a problem with pnpm install that is unable to trigger Git hooks

@Ge-yuan-jun
Copy link

Hi, all who met this problem can use yorkie-pnpm

@lsc9
Copy link

lsc9 commented Aug 12, 2022

pnpm add -D yorkie-pnpm
still not installed git hooks。

@aszx-fyh
Copy link

mark

@MissFlower
Copy link

I also encountered the same problem, did not trigger Git hooks when pnpm installed yorkie

@lifegit
Copy link

lifegit commented May 24, 2023

pnpm add -D yorkie-pnpm 仍然没有安装 git hooks。

yes,I run pnpm i yorkie-pnpm -w It's also failure

@guixianleng
Copy link

pnpm add -D yorkie-pnpm 仍然没有安装 git hooks。

yes,I run pnpm i yorkie-pnpm -w It's also failure

try it run node node_modules/yorkie-pnpm/bin/install.js

@guixianleng
Copy link

pnpm add -D yorkie-pnpm 仍然没有安装 git hooks。

yes,I run pnpm i yorkie-pnpm -w It's also failure

try it run node node_modules/yorkie-pnpm/bin/install.js

chalk version need 4.1.2

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.