Skip to content

Support Yarn Plug'n'Play#3

Merged
jscheid merged 1 commit intojscheid:masterfrom
asbish:support-yarn-pnp
May 16, 2020
Merged

Support Yarn Plug'n'Play#3
jscheid merged 1 commit intojscheid:masterfrom
asbish:support-yarn-pnp

Conversation

@asbish
Copy link
Copy Markdown
Collaborator

@asbish asbish commented Mar 18, 2020

#1

In regards to Yarn Plug'n'Play, I've selected to use setup() function in .pnp.js instead of yarn node because locate-dominating-file is expensive in tramp-mode. However setup() has side effect of enabling to require module in zip archive (patch to fs), so I've changed prettier-processes that will be able to keep node processes per project as much as possible.

By the way, I tried prettier in tramp-mode for the first time and got Error: ENOENT: no such file or directory, open '/tmp/prettier-emacs.71P4GP'. I think it maybe happens only in my environments so I replaced make-temp-file with make-nearby-temp-file and then it works fine.

@jscheid
Copy link
Copy Markdown
Owner

jscheid commented May 16, 2020

Hi, sorry I didn't get back to you about this earlier, as you know the last few months have been... interesting.

This looks great, I'm going to merge it and use it myself for a little while. There are also a few other improvements I want to make (unrelated to this) but I'm aiming to do a new release in a week or so. Thanks again!

PS can we close #1 now?

@jscheid jscheid merged commit 0e8d25b into jscheid:master May 16, 2020
@jscheid
Copy link
Copy Markdown
Owner

jscheid commented May 19, 2020

This has been released in v0.8.0, thanks again!

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