Skip to content

chore: externalize configure-nodejs dogfood coverage#2

Merged
huntharo merged 3 commits into
mainfrom
codex/dogfood-install-primer
Apr 10, 2026
Merged

chore: externalize configure-nodejs dogfood coverage#2
huntharo merged 3 commits into
mainfrom
codex/dogfood-install-primer

Conversation

@huntharo
Copy link
Copy Markdown
Contributor

@huntharo huntharo commented Apr 10, 2026

Summary

  • I switched the action to explicit cache restore/save steps so it only saves when a node_modules directory actually exists
  • I clarified that package-manager is an optional override and the default behavior follows the detected lockfile metadata
  • I moved the dogfood CI workflow, fixtures, and helper-script unit tests out of this repo into pwrdrvr/configure-nodejs-test so the published action repo stays leaner

Verification

  • I verified the action metadata YAML parses locally with Ruby's YAML loader
  • Dogfood coverage now lives in pwrdrvr/configure-nodejs-test and will exercise the action from a separate harness repository

@huntharo huntharo changed the title ci: dogfood configure-nodejs in cache primer flow chore: externalize configure-nodejs dogfood coverage Apr 10, 2026
@huntharo huntharo merged commit 69d803b into main Apr 10, 2026
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.

1 participant