Skip to content

Comments

[WIP] Fix exit code check for CLI failure handling#7012

Closed
Copilot wants to merge 2 commits intofix/cli-exit-codefrom
copilot/sub-pr-7010
Closed

[WIP] Fix exit code check for CLI failure handling#7012
Copilot wants to merge 2 commits intofix/cli-exit-codefrom
copilot/sub-pr-7010

Conversation

Copy link
Contributor

Copilot AI commented Jan 22, 2026

  • Understand the feedback from @Timeless0911 about checking exit code
  • Modify test to capture error object outside try-catch
  • Add explicit failure if execCliSync doesn't throw
  • Verify exit code (status) and stderr message after try-catch
  • Run test to confirm it works correctly

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI requested a review from Timeless0911 January 22, 2026 09:10
Copilot stopped work on behalf of Timeless0911 due to an error January 22, 2026 09:10
@chenjiahan chenjiahan deleted the copilot/sub-pr-7010 branch January 22, 2026 09:16
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