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.

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

chore(deps): Update vite to 5.2.8 #10427

Merged
merged 2 commits into from
Apr 7, 2024
Merged

Conversation

Tobbe
Copy link
Member

@Tobbe Tobbe commented Apr 7, 2024

Bumping vite from 5.1.x to 5.2.y

This will let us simplify our vite build configs. See #10220

Vite changed which export gets precedence when you have two exports of the same name. (I checked their release logs, but there is no mention of this)

These are the exports react-hook-form have:

Controller,
Form,
appendErrors,
useController,
useFieldArray,
useForm,
useFormContext,
FormProvider,
useFormState,
useWatch,
get,
set,

Plus a whole bunch of type exports.

RW also exports Form, and previously RW's form would get precedence, but with this version of vite we'd export the one from react-hook-form instead, which breaks tests (and users projects).

The fix is to move the export from rhf to before the export of RW's Form

@Tobbe Tobbe added the release:chore This PR is a chore (means nothing for users) label Apr 7, 2024
@Tobbe Tobbe added this to the next-release-patch milestone Apr 7, 2024
@Tobbe Tobbe added the changesets-ok Override the changesets check label Apr 7, 2024
@Tobbe Tobbe merged commit bc5781b into redwoodjs:main Apr 7, 2024
46 checks passed
@Tobbe Tobbe deleted the tobbe-vite-528 branch April 7, 2024 16:06
dac09 added a commit to dac09/redwood that referenced this pull request Apr 9, 2024
…auth-provider-p1

* 'main' of github.com:redwoodjs/redwood:
  fix(middleware): Handle POST requests in middleware router too (redwoodjs#10418)
  chore(ci): get ci running on next (redwoodjs#10432)
  RSC: Explain noExternal vite config option (redwoodjs#10429)
  chore(web): Fix .d.ts overwrite build issue (redwoodjs#10431)
  chore(web): .js imports to prep for ESM (redwoodjs#10430)
  chore(refactor): Split rwjs/forms up into several smaller logical units (redwoodjs#10428)
  chore(rsc): simplify `noExternals` config (redwoodjs#10220)
  chore(deps): Update vite to 5.2.8 (redwoodjs#10427)
  chore(auth): Convert `@redwoodjs/auth` to ESM+CJS dual build (redwoodjs#10417)
  chore(framework-tools): Warn about missing metafile (redwoodjs#10426)
  chore(test): Switch rwjs/auth over to vitest (redwoodjs#10423)
  chore(whatwg-fetch): Switch to importing instead of requiring (redwoodjs#10424)
  chore(deps): bump undici from 5.28.3 to 5.28.4 in /.github/actions/check_changesets (redwoodjs#10421)
  chore(route-manifest): Add relativeFilePath to route manifest (redwoodjs#10416)
@jtoar
Copy link
Contributor

jtoar commented Apr 9, 2024

This builds on the Vite v5 upgrade which is slated for the next major. See #10197.

dac09 added a commit that referenced this pull request Apr 11, 2024
…th-mw-auth

* 'main' of github.com:redwoodjs/redwood: (21 commits)
  fix(auth): Handle when authorization header is lowercased (#10442)
  Update rbac.md - code match (#10405)
  chore: make crwa e2e test work across branches (#10437)
  feat: [Auth] Common AuthProvider & use* changes for middleware auth (#10420)
  fix(cli): only show webpack options for dev if `bundler = "webpack"` (#10359)
  fix(vercel): specify build env vars as a string (#10436)
  fix(vercel): write `vercel.json` as a part of setup (#10355)
  fix(middleware): Handle POST requests in middleware router too (#10418)
  chore(ci): get ci running on next (#10432)
  RSC: Explain noExternal vite config option (#10429)
  chore(web): Fix .d.ts overwrite build issue (#10431)
  chore(web): .js imports to prep for ESM (#10430)
  chore(refactor): Split rwjs/forms up into several smaller logical units (#10428)
  chore(rsc): simplify `noExternals` config (#10220)
  chore(deps): Update vite to 5.2.8 (#10427)
  chore(auth): Convert `@redwoodjs/auth` to ESM+CJS dual build (#10417)
  chore(framework-tools): Warn about missing metafile (#10426)
  chore(test): Switch rwjs/auth over to vitest (#10423)
  chore(whatwg-fetch): Switch to importing instead of requiring (#10424)
  chore(deps): bump undici from 5.28.3 to 5.28.4 in /.github/actions/check_changesets (#10421)
  ...
@Josh-Walker-GM Josh-Walker-GM added release:dependency This PR only updates dependencies and removed release:chore This PR is a chore (means nothing for users) labels Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changesets-ok Override the changesets check release:dependency This PR only updates dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants