Skip to content

fix: clarify architecture warning without version#48

Closed
cnYui wants to merge 1 commit into
vlang:mainfrom
cnYui:fix/v-architecture-warning
Closed

fix: clarify architecture warning without version#48
cnYui wants to merge 1 commit into
vlang:mainfrom
cnYui:fix/v-architecture-warning

Conversation

@cnYui

@cnYui cnYui commented Jul 15, 2026

Copy link
Copy Markdown

Fixes #26.

Summary

  • update the architecture without version warning so it describes V setup behavior instead of Node.js
  • clarify that this path installs V for the requested architecture from the default branch and skips versioned caching
  • rebuild the checked-in action output

Verification

  • npm test (23 passed, 5 existing Windows baseline failures; the same five failures reproduce on the unmodified 663cc08827d1fc0532ee755b4dedbb183c11d06a baseline)
  • npm run build
  • npm run lint
  • npx prettier --check "**/*.ts"
  • npm run package
  • git diff --check

Note: npm run format-check is not usable as-is in PowerShell because the single-quoted glob is passed literally to Prettier as "'**/*.ts'". I ran the equivalent double-quoted Prettier command above.

@ulises-jeremias

Copy link
Copy Markdown
Member

Thanks @cnYui for the improved warning message! This PR and #38 both address the same issue (#26 — Node copy-paste leftover). #38 is more comprehensive as it also fixes the architecture input description in action.yml (#27). I'll merge #38 and close this to avoid duplicate dist/index.js conflicts.

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.

fix: Node copy-paste leftover in architecture warning message

2 participants