Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Use prettier_d #194

Closed
zimme opened this issue Jun 14, 2017 · 4 comments
Closed

Use prettier_d #194

zimme opened this issue Jun 14, 2017 · 4 comments

Comments

@zimme
Copy link
Member

zimme commented Jun 14, 2017

I think prettier_d would be a really good match for this project.

Any thoughts?

@darahak
Copy link
Collaborator

darahak commented Jun 14, 2017

Not 100% sure, but aren't Atom packages (including dependencies) cached after activation?
The point of prettier_d is to mitigate startup time, which should already be low in Atom.

@zimme
Copy link
Member Author

zimme commented Jun 14, 2017

// charypar: This is currently the best way to use local prettier instance.
// Using the CLI introduces a noticeable delay and there is currently no
// way to use prettier as a long-running process for formatting files as needed
//
// See prettier/prettier#918

~~This comment from here makes me think otherwise.~~~

Scratch that, I misunderstood the comment.

@josephfrazier
Copy link

prettier_d author here (by way of prettier/prettier#918). I'd love to see prettier-atom use prettier_d to speed things up, if prettier-atom doesn't already cache loaded prettier instances. Let me know if you have any questions!

@robwise
Copy link
Collaborator

robwise commented Sep 16, 2017

I'm going to go ahead and close this because we are already caching the lookup path using atom-linter tools' findCached

@robwise robwise closed this as completed Sep 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants