Skip to content

Conversation

markdalgleish
Copy link
Member

@markdalgleish markdalgleish commented Mar 13, 2025

This is backporting a fix from @react-router/dev that ensures that the getAppDirectory function in routes.ts works when there are multiple copies of @remix-run/dev.

Note that this doesn't fix the issue if any of the resolved versions of @remix-run/dev don't contain this fix, but this will ensure that this issue doesn't crop up with any future releases.

Copy link

changeset-bot bot commented Mar 13, 2025

🦋 Changeset detected

Latest commit: 2f20c83

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

This PR includes changesets to release 19 packages
Name Type
@remix-run/dev Patch
@remix-run/fs-routes Patch
@remix-run/route-config Patch
@remix-run/routes-option-adapter Patch
create-remix Patch
remix Patch
@remix-run/architect Patch
@remix-run/cloudflare Patch
@remix-run/cloudflare-pages Patch
@remix-run/cloudflare-workers Patch
@remix-run/css-bundle Patch
@remix-run/deno Patch
@remix-run/eslint-config Patch
@remix-run/express Patch
@remix-run/node Patch
@remix-run/react Patch
@remix-run/serve Patch
@remix-run/server-runtime Patch
@remix-run/testing 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

@brophdawg11 brophdawg11 merged commit 290c189 into release-next Mar 13, 2025
8 of 9 checks passed
@brophdawg11 brophdawg11 deleted the markdalgleish/routes-handle-multiple-dev-packages branch March 13, 2025 16:54
Copy link
Contributor

🤖 Hello there,

We just published version 2.16.1-pre.1 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!

Copy link
Contributor

🤖 Hello there,

We just published version 2.16.1 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.

2 participants