Skip to content

Conversation

@amareshsm
Copy link
Member

Prerequisites checklist

What is the purpose of this pull request?

Fix build warnings:
image

What changes did you make? (Give an overview)

  • add Vite bundle optimisation with vendor/parser/ui chunk splitting for better caching and load time.
Scenario Before (entire bundle changes) After (specific chunk only changes)
First Visit 5.8 MB 5.8 MB
Code Update 5.8 MB 263 KB
UI Library Update 5.8 MB 116 KB
Dependency Update 5.8 MB 18 KB
  • Safelist Migration
    Before (Tailwind v2 syntax - deprecated)
purge: {
  content: ["./{pages,components,app,src}/**/*.{ts,tsx}"],
  options: {
    safelist: ["cm-editor", "cm-gutter", "ͼ1", "cm-focused"],
  },
},

After (Tailwind v3+ syntax - current)

safelist: ["cm-editor", "cm-gutter", "ͼ1", "cm-focused"],

Related Issues

Is there anything you'd like reviewers to focus on?

@netlify
Copy link

netlify bot commented Oct 15, 2025

Deploy Preview for eslint-code-explorer ready!

Name Link
🔨 Latest commit bef992d
🔍 Latest deploy log https://app.netlify.com/projects/eslint-code-explorer/deploys/68f007afb0c4dc00083b9629
😎 Deploy Preview https://deploy-preview-191--eslint-code-explorer.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 project configuration.

@eslintbot eslintbot added this to Triage Oct 15, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Oct 15, 2025
@aladdin-add aladdin-add moved this from Needs Triage to Implementing in Triage Oct 17, 2025
@aladdin-add aladdin-add self-requested a review October 17, 2025 02:44
@aladdin-add aladdin-add added the accepted There is consensus among the team that this change meets the criteria for inclusion label Oct 17, 2025
@lumirlumir lumirlumir self-requested a review October 17, 2025 10:12
Copy link
Member

@lumirlumir lumirlumir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like a nice optimization and works well on my end.

I've left a single comment about an unused class name. Otherwise, everything looks good.

Copy link
Member

@kecrily kecrily left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks

Copy link
Member

@lumirlumir lumirlumir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@lumirlumir lumirlumir merged commit fdf7814 into main Oct 20, 2025
9 checks passed
@lumirlumir lumirlumir deleted the optimize-the-build branch October 20, 2025 09:46
@github-project-automation github-project-automation bot moved this from Implementing to Complete in Triage Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted There is consensus among the team that this change meets the criteria for inclusion build

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

5 participants