bigcommerce / catalyst

Catalyst - for Composable Commerce
https://catalyst-demo.site
MIT License
91 stars 64 forks source link

Speed up builds by excluding node_modules from tailwind config #1236

Closed bookernath closed 1 month ago

bookernath commented 1 month ago

What/Why?

https://github.com/vercel/next.js/issues/37825#issuecomment-1633364694

Our build time was spending over 60 seconds on Creating an optimized production build... so I tried removing the node_modules folder from the content in tailwind config, and I see much faster build times now.

Testing

Preview build

Before:

image image

After:

image image
changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: ed371e07fb21762599c5647c5d15eb05d938cc38

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

This PR includes changesets to release 1 package | Name | Type | | -------------------------- | ----- | | @bigcommerce/catalyst-core | 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

vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
catalyst-latest ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 7, 2024 2:14am
6 Skipped Deployments | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **catalyst** | ⬜️ Ignored ([Inspect](https://vercel.com/bigcommerce-platform/catalyst/2XtnQ3Kv76baAaMDc3Yr96H9NRwB)) | | | Aug 7, 2024 2:14am | | **catalyst-1millionproducts-store** | ⬜️ Ignored ([Inspect](https://vercel.com/bigcommerce-platform/catalyst-1millionproducts-store/6c9QYK24tgJoqoLitaqiu7vLaUbR)) | [Visit Preview](https://catalyst-1millionproducts-store-git-9b94d7-bigcommerce-platform.vercel.app) | | Aug 7, 2024 2:14am | | **catalyst-au** | ⬜️ Ignored ([Inspect](https://vercel.com/bigcommerce-platform/catalyst-au/GgvFaMeZ3ez87xsBQauUoGP1XYCr)) | [Visit Preview](https://catalyst-au-git-tailwind-build-bigcommerce-platform.vercel.app) | | Aug 7, 2024 2:14am | | **catalyst-test-store** | ⬜️ Ignored ([Inspect](https://vercel.com/bigcommerce-platform/catalyst-test-store/2nWM9nAAhHsEveyHCYevY5Xib9V9)) | [Visit Preview](https://catalyst-test-store-git-tailwind-build-bigcommerce-platform.vercel.app) | | Aug 7, 2024 2:14am | | **catalyst-uk** | ⬜️ Ignored ([Inspect](https://vercel.com/bigcommerce-platform/catalyst-uk/EjPhyniXe96WZkNpsAtPzyyR6cY8)) | [Visit Preview](https://catalyst-uk-git-tailwind-build-bigcommerce-platform.vercel.app) | | Aug 7, 2024 2:14am | | **catalyst-unstable** | ⬜️ Ignored ([Inspect](https://vercel.com/bigcommerce-platform/catalyst-unstable/6epy3EnrWfzCVSdbot2g1exKFdkz)) | [Visit Preview](https://catalyst-unstable-git-tailwind-build-bigcommerce-platform.vercel.app) | | Aug 7, 2024 2:14am |
github-actions[bot] commented 1 month ago

⚡️🏠 Lighthouse report

Lighthouse ran against https://catalyst-latest-q16lo7wbf-bigcommerce-platform.vercel.app

🖥️ Desktop

We ran Lighthouse against the changes on a desktop and produced this report. Here's the summary:

Category Score
🟠 Performance 85
🟢 Accessibility 100
🟢 Best practices 96
🟠 SEO 82

📱 Mobile

We ran Lighthouse against the changes on a mobile and produced this report. Here's the summary:

Category Score
🟢 Performance 92
🟢 Accessibility 100
🟢 Best practices 96
🟠 SEO 85