Skip to content

fix: add corepack enable to GitHub Actions workflows - #8

Merged
rohit-gohri merged 6 commits into
mainfrom
copilot/fix-missing-corepack-github-actions
Mar 30, 2026
Merged

rohit-gohri merged 6 commits into
mainfrom
copilot/fix-missing-corepack-github-actions

Conversation

Copilot AI commented Mar 30, 2026 •

Copy link
Copy Markdown
Contributor
  • Diagnose: tests reference non-existent hello/hello world commands; actual commands are patch, commit, init
  • Diagnose existing bug: patch and commit commands use callback-style dir() without await, causing run() to return before async work finishes
  • Fix src/commands/patch/index.ts: wrap dir() callback in a Promise so run() properly awaits
  • Fix src/commands/commit/index.ts: same fix
  • Remove stale test/commands/hello/ test files
  • Add test/commands/patch/index.test.ts for the patch command
  • Add test/commands/commit/index.test.ts for the commit command
  • Add test/commands/init/index.test.ts for the init command
  • All 7 tests pass, lint clean, CodeQL clean

Copilot AI and others added 2 commits March 30, 2026 22:12
Comment thread .github/workflows/onPushToMain.yml Outdated
Comment thread .github/workflows/onRelease.yml Outdated
Comment thread .github/workflows/test.yml Outdated
Co-authored-by: Rohit Gohri <code@rohit.page>
@rohit-gohri
rohit-gohri marked this pull request as ready for review March 30, 2026 23:52
Comment thread .github/workflows/onPushToMain.yml Outdated
Comment thread .github/workflows/onRelease.yml Outdated
Comment thread .github/workflows/test.yml Outdated
Co-authored-by: Rohit Gohri <code@rohit.page>
@rohit-gohri
rohit-gohri merged commit 0771e33 into main Mar 30, 2026
6 checks passed
@rohit-gohri
rohit-gohri deleted the copilot/fix-missing-corepack-github-actions branch March 30, 2026 23:55
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