Skip to content

remove unused browserslist config - #751

Open
mpurnell1 wants to merge 1 commit into
commaai:masterfrom
mpurnell1:rm-browserslist
Open

mpurnell1 wants to merge 1 commit into
commaai:masterfrom
mpurnell1:rm-browserslist

Conversation

@mpurnell1

Copy link
Copy Markdown

Nothing in the build reads the browserslist block: Vite uses build.target, Tailwind v4 dropped browserslist support, and the only package in the tree that can read it (@babel/core, via @vitejs/plugin-react) is run with config loading disabled and is skipped entirely in production builds. The production dist/ built from this branch is byte-identical to master's.

The changes in this PR were generated by Claude Opus 5, then human reviewed; lint, tests and a production build pass locally with bun 1.3.14.

Vite, @vitejs/plugin-react and Tailwind v4 all ignore it; Vite uses its
default build.target.

Assisted-by: Claude
@github-actions

Copy link
Copy Markdown

Welcome to connect! Make sure to:

  • read the contributing guidelines
  • mark your PR as a draft until it's ready to review
  • post the preview on Discord; feedback from users will speedup the PR review

deployed preview: https://751.connect-d5y.pages.dev
gallery report: https://751.connect-d5y.pages.dev/connect-gallery

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant