Skip to content

feat: attempt to enable v8 compiler caching #1059

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

Merged
merged 4 commits into from
Apr 28, 2025
Merged

feat: attempt to enable v8 compiler caching #1059

merged 4 commits into from
Apr 28, 2025

Conversation

tivac
Copy link
Owner

@tivac tivac commented Apr 28, 2025

Description

If available, use the node v22 API to enable the v8 compiler cache.

Motivation and Context

ESLint saw improvements of 90% in startup times, we want in on that.

https://bsky.app/profile/humanwhocodes.com/post/3lnvcvv5p6a25

How Has This Been Tested?

Normal tests all passing, including in playwright

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have added a changeset for my change.

Copy link

changeset-bot bot commented Apr 28, 2025

🦋 Changeset detected

Latest commit: 475dc17

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 10 packages
Name Type
@modular-css/processor
@modular-css/rollup Patch
@modular-css/browserify Patch
@modular-css/cli Patch
@modular-css/css-to-js Patch
@modular-css/glob Patch
@modular-css/postcss Patch
@modular-css/svelte Patch
@modular-css/vite Patch
@modular-css/webpack Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

netlify bot commented Apr 28, 2025

Deploy Preview for m-css ready!

Name Link
🔨 Latest commit 475dc17
🔍 Latest deploy log https://app.netlify.com/sites/m-css/deploys/680fcd29b40ab800089294fc
😎 Deploy Preview https://deploy-preview-1059--m-css.netlify.app
📱 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 configuration.

tivac added 2 commits April 28, 2025 11:06
Looks like rollup changed circular dependency handling
@tivac tivac merged commit 46b1362 into main Apr 28, 2025
9 checks passed
@tivac tivac deleted the compile-cache branch April 28, 2025 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant