Skip to content

ci: fix setup-node in canary workflow#1840

Merged
erickzhao merged 1 commit intomainfrom
ci-fix-canary-workflow
Oct 3, 2025
Merged

ci: fix setup-node in canary workflow#1840
erickzhao merged 1 commit intomainfrom
ci-fix-canary-workflow

Conversation

@dsanders11
Copy link
Member

Summarize your changes:

We accidentally broke the canary workflow in #1821 because we only check out electron/minimal-repro there, and that repo doesn't have a .nvmrc file.

It probably also makes sense for the canary workflow to be using the latest Node.js anyway to catch any potential errors.

@dsanders11 dsanders11 requested a review from a team as a code owner October 3, 2025 19:12
@dsanders11 dsanders11 requested a review from erickzhao October 3, 2025 19:13
@erickzhao erickzhao merged commit b7cf4e8 into main Oct 3, 2025
7 checks passed
@erickzhao erickzhao deleted the ci-fix-canary-workflow branch October 3, 2025 19:58
@continuous-auth
Copy link

🎉 This PR is included in version 19.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants