Skip to content

fix: specify node version in package.json#1107

Merged
mbifulco merged 2 commits into
mainfrom
fix/workflow-versions
Nov 10, 2025
Merged

fix: specify node version in package.json#1107
mbifulco merged 2 commits into
mainfrom
fix/workflow-versions

Conversation

@mbifulco

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings November 10, 2025 13:00
@vercel

vercel Bot commented Nov 10, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
mikebifulco-com-bnbu Ready Ready Preview Comment Nov 10, 2025 1:05pm

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR migrates Node.js version configuration from .nvmrc to the engines field in package.json, centralizing version management and ensuring consistency across GitHub Actions workflows.

Key changes:

  • Removed .nvmrc file
  • Added engines field to package.json specifying Node 24 and pnpm 10
  • Updated all GitHub Actions workflows to read Node version from package.json instead of .nvmrc or hardcoded lts/*

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
package.json Added engines field specifying Node.js and pnpm versions
.nvmrc Removed in favor of package.json engines field
.github/workflows/test-unit.yml Updated to read Node version from package.json
.github/workflows/test-e2e.yml Updated to read Node version from package.json
.github/workflows/sync-newsletter.yml Updated setup-node to v6 and changed to read Node version from package.json
.github/workflows/lint.yml Updated to read Node version from package.json

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mbifulco mbifulco merged commit a4669c2 into main Nov 10, 2025
8 checks passed
@mbifulco mbifulco deleted the fix/workflow-versions branch November 10, 2025 13:05
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