Skip to content

fix(ci): fix fnm and nvm-windows migration tests#149

Merged
CalvinAllen merged 1 commit intomainfrom
fix/ci/fnm-nvm-windows-migration-tests
Dec 14, 2025
Merged

fix(ci): fix fnm and nvm-windows migration tests#149
CalvinAllen merged 1 commit intomainfrom
fix/ci/fnm-nvm-windows-migration-tests

Conversation

@CalvinAllen
Copy link
Contributor

Summary

Fix the remaining Node.js migration test failures:

fnm (macOS):

  • Change selection from 1 to all - same issue as nvm (Ubuntu)
  • System Node.js was being selected instead of fnm-installed version

nvm-windows (Windows):

  • Add nvm directories to GITHUB_PATH for cross-step availability
  • The PATH refresh wasn't working across PowerShell steps
  • Change selection from 1 to all

Test plan

  • Verify migrate-node-macos-fnm workflow passes
  • Verify migrate-node-windows-nvm workflow passes

fnm (macOS):
- Change selection from "1" to "all" to migrate all detected versions

nvm-windows (Windows):
- Add nvm directories to GITHUB_PATH for cross-step availability
- Change selection from "1" to "all" to migrate all detected versions
@CalvinAllen CalvinAllen merged commit 5f1b6b9 into main Dec 14, 2025
2 checks passed
@CalvinAllen CalvinAllen deleted the fix/ci/fnm-nvm-windows-migration-tests branch December 14, 2025 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant