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

feat(express): support Express v5 #13064

Merged
merged 2 commits into from
Feb 20, 2025
Merged

feat(express): support Express v5 #13064

merged 2 commits into from
Feb 20, 2025

Conversation

rossipedia
Copy link
Contributor

express@5 is out, and is mostly compatible with express@4 (see the migration guide). At least for React Router usage the differences should be immaterial.

This would allow users to integrate React Router into their existing express@5 apps, or use express@5 as a hosting server, without having to fiddle with --legacy-peer-deps or similar.

Copy link

changeset-bot bot commented Feb 19, 2025

⚠️ No Changeset found

Latest commit: b926f2d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@remix-cla-bot
Copy link
Contributor

remix-cla-bot bot commented Feb 19, 2025

Hi @rossipedia,

Welcome, and thank you for contributing to React Router!

Before we consider your pull request, we ask that you sign our Contributor License Agreement (CLA). We require this only once.

You may review the CLA and sign it by adding your name to contributors.yml.

Once the CLA is signed, the CLA Signed label will be added to the pull request.

If you have already signed the CLA and received this response in error, or if you have any questions, please contact us at [email protected].

Thanks!

- The Remix team

@MichaelDeBoey MichaelDeBoey changed the title [express] Add express@5 to peerDependencies feat(express): support Express v5 Feb 19, 2025
@rossipedia rossipedia changed the base branch from main to dev February 19, 2025 23:06
@remix-cla-bot
Copy link
Contributor

remix-cla-bot bot commented Feb 19, 2025

Thank you for signing the Contributor License Agreement. Let's get this merged! 🥳

@rossipedia rossipedia force-pushed the patch-1 branch 2 times, most recently from 4ab5e78 to 4464d60 Compare February 19, 2025 23:07
@MichaelDeBoey
Copy link
Member

@rossipedia It seems like something went wrong when rebasing onto dev, could you please remove all commits except yours?

`express@5` is out, and is mostly compatible with `express@4` (see [the migration guide](https://expressjs.com/en/guide/migrating-5.html)). At least for React Router usage the differences should be immaterial.

This would allow users to integrate React Router into their existing `express@5` apps, or use `express@5` as a hosting server, without having to fiddle with `--legacy-peer-deps` or similar.
@rossipedia
Copy link
Contributor Author

@MichaelDeBoey I think that should do it, does it look OK now?

@timdorr
Copy link
Member

timdorr commented Feb 20, 2025

Thank you!

@timdorr timdorr merged commit b0756d2 into remix-run:dev Feb 20, 2025
5 checks passed
brophdawg11 added a commit that referenced this pull request Feb 20, 2025
wilcoxmd added a commit to wilcoxmd/react-router that referenced this pull request Mar 5, 2025
…d-route-typegen

* upstream/dev: (65 commits)
  Generate types for `virtual:react-router/server-build` (remix-run#13152)
  Add support for client context and middleware (unstable) (remix-run#12941)
  Add playground for `vite-plugin-cloudflare` (remix-run#13151)
  do not typegen params for layout routes with a corresponding index (remix-run#13131) (remix-run#13140)
  Fix types for `loaderData` and `actionData` that contain `Record`s (remix-run#13139)
  chore: format
  chore(dev): remove unused dependencies (remix-run#13134)
  Remove unused Vite file system watcher (remix-run#13133)
  Remove stale changesets cherry-picked into release-next for 7.2.0
  Fix custom SSR build input with `serverBundles` (remix-run#13107)
  Skip resource route flow in dev mode when SPA  mode is enabled (remix-run#13113)
  chore: format
  Add integration test for `vite-plugin-cloudflare` (remix-run#13099)
  Fix custom client `build.rollupOptions.output.entryFileNames` (remix-run#13098)
  Detect lazy route discovery manifest version mismatches and trigger reloads (remix-run#13061)
  Fix critical CSS with custom `Vite.DevEnvironment` (remix-run#13066)
  Fix usage of `prerender` option with `serverBundles` (remix-run#13082)
  Fix support for custom `build.assetsDir` (remix-run#13077)
  Add changeset for remix-run#13064
  Only import the root route when SSRing SPA mode's index.html (remix-run#13023)
  ...
Copy link
Contributor

github-actions bot commented Mar 6, 2025

🤖 Hello there,

We just published version 7.3.0 which includes this pull request. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

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.

3 participants