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.

Sign up for GitHub

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

Jump to bottom

Try using v8-compile-cache #3306

Closed
KyleAMathews opened this issue Dec 21, 2017 · 7 comments
Closed

Try using v8-compile-cache #3306

KyleAMathews opened this issue Dec 21, 2017 · 7 comments
Labels
good first issue Issue that doesn't require previous experience with Gatsby help wanted Issue with a clear description that the community can help with.

Comments

@KyleAMathews
Copy link
Contributor

webpack-cli just added it and saw substantial speedups

https://twitter.com/ev1stensberg/status/943865463406301184

https://www.npmjs.com/package/v8-compile-cache
@KyleAMathews KyleAMathews added good first issue Issue that doesn't require previous experience with Gatsby Feature Request help wanted Issue with a clear description that the community can help with. labels Dec 21, 2017
@KyleAMathews
Copy link
Contributor Author

Easy to add — just require it webpack/webpack-cli@0564ceb

@pwener
Copy link
Contributor

pwener commented Dec 22, 2017 •
edited
Loading

I can do?

@KyleAMathews
Copy link
Contributor Author

@pwener would love a PR! Please test the bootstrap times with and without the compile cache.

@pwener
Copy link
Contributor

pwener commented Dec 24, 2017

Hello, where you think would be the best place to put this? I've try with gatsby/packages/gatsby/src/bin/gatsby.js but the bootstrap times decreased little.

@KyleAMathews
Copy link
Contributor Author

It might not have much effect. Try adding a process.exit() right after the require and measure the time to their with and without the require.

@pwener pwener mentioned this issue Dec 26, 2017
Merged
@pwener
Copy link
Contributor

pwener commented Dec 27, 2017 •
edited
Loading

The results were good, can you evaluate my PR?

@KyleAMathews
Copy link
Contributor Author

PR merged 💯

This was referenced Feb 20, 2022
Open
Open
Open
Open
Open
@monizb monizb mentioned this issue Feb 16, 2023
Merged
@0xSebin 0xSebin mentioned this issue Feb 28, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Issue that doesn't require previous experience with Gatsby help wanted Issue with a clear description that the community can help with.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@KyleAMathews @pwener