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(remix-vercel): add support for @vercel/node v2 #3420

Merged
merged 15 commits into from
Jul 12, 2022
Merged

Conversation

mcansh
Copy link
Collaborator

@mcansh mcansh commented Jun 8, 2022

Closes: #

  • Docs
  • Tests

Testing Strategy:

@mcansh mcansh closed this Jun 8, 2022
@mcansh mcansh deleted the mcansh-patch-1 branch June 8, 2022 17:38
@MichaelDeBoey
Copy link
Member

We already have #3412

@MichaelDeBoey MichaelDeBoey added duplicate This issue or pull request already exists adapter:vercel labels Jun 8, 2022
@mcansh
Copy link
Collaborator Author

mcansh commented Jun 8, 2022

yup didnt see it 🙈

@MichaelDeBoey MichaelDeBoey changed the title chore(remix-vercel): bump @vercel/node peerDep version feat(remix-vercel): add support for @vercel/node v2 Jun 8, 2022
@MichaelDeBoey MichaelDeBoey restored the mcansh-patch-1 branch June 8, 2022 19:09
@MichaelDeBoey
Copy link
Member

MichaelDeBoey commented Jun 8, 2022

@mcansh We have permission from @styfle to not wait for approval of his employer and re-open & merge this one

I think we should update @vercel/node in devDependencies too though.

@MichaelDeBoey MichaelDeBoey reopened this Jun 8, 2022
@MichaelDeBoey MichaelDeBoey removed the duplicate This issue or pull request already exists label Jun 8, 2022
@MichaelDeBoey
Copy link
Member

@mcansh Seems like build & tests are failing. Probably need to re-generate yarn.lock

@mcansh
Copy link
Collaborator Author

mcansh commented Jun 8, 2022

@mcansh Seems like build & tests are failing. Probably need to re-generate yarn.lock

caused by our test using an internal vercel/node module, looking into it

Signed-off-by: Logan McAnsh <logan@mcan.sh>
@MichaelDeBoey
Copy link
Member

Seems like vercel/vercel#7931 is causing the build to fail 😕

@mcansh
Copy link
Collaborator Author

mcansh commented Jun 8, 2022

looks like @vercel/node uses @vercel/build-utils but it's not in its regular dependencies 👀

yup, like you said 🙈

@MichaelDeBoey
Copy link
Member

MichaelDeBoey commented Jun 8, 2022

Created vercel/vercel#7934 to fix it

CC/ @styfle since you're on the Vercel team, you can probably merge that for us please?

@styfle
Copy link

styfle commented Jun 8, 2022

How about installing @vercel/build-utils to your devDependencies?

@MichaelDeBoey
Copy link
Member

@styfle It's used by @vercel/node, so that package should have it in dependencies

Copy link
Member

@MichaelDeBoey MichaelDeBoey left a comment

Choose a reason for hiding this comment

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

@mcansh @vercel/node@2.2.0 is released now 🥳🎉

packages/remix-vercel/package.json Outdated Show resolved Hide resolved
packages/remix-vercel/package.json Outdated Show resolved Hide resolved
mcansh and others added 3 commits June 16, 2022 13:16
Co-authored-by: Michaël De Boey <info@michaeldeboey.be>
Co-authored-by: Michaël De Boey <info@michaeldeboey.be>
Signed-off-by: Logan McAnsh <logan@mcan.sh>
@MichaelDeBoey
Copy link
Member

@styfle It seems like @vercel/build-utils is missing @vercel/frameworks & @vercel/routing-utils in it's dependencies list.
https://github.com/remix-run/remix/runs/6924983939?check_suite_focus=true#step:6:124

Error: node_modules/@vercel/build-utils/dist/detect-builders.d.ts(1,23): error TS2307: Cannot find module '@vercel/routing-utils' or its corresponding type declarations.
Error: node_modules/@vercel/build-utils/dist/detect-framework.d.ts(1,27): error TS2307: Cannot find module '@vercel/frameworks' or its corresponding type declarations.
Error: node_modules/@vercel/build-utils/dist/workspaces/workspace-managers.d.ts(1,32): error TS2307: Cannot find module '@vercel/frameworks' or its corresponding type declarations.
Error: node_modules/@vercel/build-utils/dist/monorepos/monorepo-managers.d.ts(1,32): error TS2307: Cannot find module '@vercel/frameworks' or its corresponding type declarations.

I however think it would be better to remove @vercel/build-utils from @vercel/node instead tbh.

@MichaelDeBoey
Copy link
Member

Waiting for vercel/vercel#7951 (comment) to be resolved

@styfle
Copy link

styfle commented Jun 16, 2022

Can you add them as devDependencies as a temporary workaround? We probably won't be able to refactor @vercel/build-utils today since it will take thorough planning and review.

@MichaelDeBoey
Copy link
Member

@styfle We've reverted changes to templates/vercel for now and we'll wait until everything is fixed in @vercel/node before we add support for v2

@MichaelDeBoey MichaelDeBoey marked this pull request as draft June 16, 2022 21:48
Copy link
Member

@MichaelDeBoey MichaelDeBoey left a comment

Choose a reason for hiding this comment

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

@mcansh @vercel/node@2.3.3 is released now 🥳🎉

Don't forget to update yarn.lock as well.

packages/remix-vercel/package.json Outdated Show resolved Hide resolved
packages/remix-vercel/package.json Outdated Show resolved Hide resolved
mcansh and others added 4 commits July 1, 2022 11:47
Co-authored-by: Michaël De Boey <info@michaeldeboey.be>
Co-authored-by: Michaël De Boey <info@michaeldeboey.be>
Signed-off-by: Logan McAnsh <logan@mcan.sh>
@mcansh mcansh marked this pull request as ready for review July 1, 2022 15:52
@mcansh mcansh marked this pull request as draft July 1, 2022 16:41
Copy link
Member

@MichaelDeBoey MichaelDeBoey left a comment

Choose a reason for hiding this comment

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

@mcansh @vercel/node@2.4.0 is released now 🥳🎉

Don't forget to update yarn.lock as well.

packages/remix-vercel/package.json Outdated Show resolved Hide resolved
packages/remix-vercel/package.json Outdated Show resolved Hide resolved
Co-authored-by: Michaël De Boey <info@michaeldeboey.be>
@changeset-bot
Copy link

changeset-bot bot commented Jul 12, 2022

🦋 Changeset detected

Latest commit: 66a4333

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

This PR includes changesets to release 16 packages
Name Type
@remix-run/vercel Patch
remix Patch
@remix-run/dev Patch
@remix-run/eslint-config Patch
@remix-run/react Patch
@remix-run/serve Patch
@remix-run/server-runtime Patch
@remix-run/cloudflare Patch
@remix-run/node Patch
@remix-run/deno Patch
@remix-run/cloudflare-pages Patch
@remix-run/cloudflare-workers Patch
@remix-run/express Patch
@remix-run/netlify Patch
create-remix Patch
@remix-run/architect 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

mcansh and others added 4 commits July 12, 2022 10:39
@mcansh mcansh marked this pull request as ready for review July 12, 2022 14:57
Copy link
Member

@kentcdodds kentcdodds left a comment

Choose a reason for hiding this comment

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

Sweet! Thanks 👍

@github-actions
Copy link
Contributor

🤖 Hello there,

We just published version v0.0.0-nightly-3c6c798-20220713 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.

4 participants