Skip to content

Conversation

ddgromit
Copy link
Contributor

@ddgromit ddgromit commented Apr 24, 2025

Description

Lock the node & pnpm versions

CleanShot 2025-04-24 at 13 05 14@2x

Testing

  • I have tested these changes locally
  • I have run the validation scripts (pnpm run validate)
  • I have checked that the documentation builds correctly

Copy link

github-actions bot commented Apr 24, 2025

🌿 Documentation Preview

Name Status Preview Updated (UTC)
Alchemy Docs ✅ Ready 🔗 Visit Preview Jun 2, 2025, 8:29 PM

@ddgromit ddgromit requested a review from dslovinsky April 24, 2025 20:18
@github-actions github-actions bot temporarily deployed to docs-preview April 24, 2025 20:18 Destroyed
@@ -0,0 +1 @@
nodejs 22.15.0
Copy link
Contributor Author

@ddgromit ddgromit Apr 24, 2025

Choose a reason for hiding this comment

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

For asdf or mise users, keeps us on a standard Node Version. Vercel defaults to v22. This only applies locally - it won't change what vercel uses.

Comment on lines +6 to +8
"engines": {
"node": "22.x"
},
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Make explicit the node version that vercel will use (this is currently the default)

package.json Outdated
"engines": {
"node": "22.x"
},
"packageManager": "pnpm@10.9.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Lock the pnpm version for both us and vercel

@github-actions github-actions bot temporarily deployed to docs-preview April 24, 2025 20:23 Destroyed
@github-actions github-actions bot temporarily deployed to docs-preview April 24, 2025 20:28 Destroyed
@github-actions github-actions bot temporarily deployed to docs-preview April 24, 2025 20:34 Destroyed
Copy link
Collaborator

@dslovinsky dslovinsky left a comment

Choose a reason for hiding this comment

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

@ddgromit Will take a closer look at this later but FYI this isn't deploying on our Vercel. Fern have their own setup it deploys on. I believe it uses Vercel under the hood but sounds like its super custom

@ddgromit
Copy link
Contributor Author

Ah I see, so if they are just hosting the build/ dir then they won't read these values anyway. Still useful for local dev and CI IMO.

uses: pnpm/action-setup@v4
id: pnpm-install
with:
version: 9
Copy link
Collaborator

Choose a reason for hiding this comment

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

We should specify this no? I believe it doesn't read package.json

@dslovinsky dslovinsky requested a review from a team as a code owner June 2, 2025 20:26
@github-actions github-actions bot temporarily deployed to docs-preview June 2, 2025 20:27 Destroyed
@dslovinsky dslovinsky merged commit a2be24d into main Jun 2, 2025
8 checks passed
@dslovinsky dslovinsky deleted the derek/versioning branch June 2, 2025 20:29
dslovinsky added a commit that referenced this pull request Jun 5, 2025
* Versions and corepack

* Vercel engine

* Github action

* CI versions

* Version file

---------

Co-authored-by: dslovinsky <dslovinsky@users.noreply.github.com>
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.

2 participants