Skip to content

chore: attribute exit code 127 as a missing executable - #1998

Merged
aws-cdk-automation merged 2 commits into
mainfrom
gandhya-canary-errormessage
Sep 28, 2026
Merged

aws-cdk-automation merged 2 commits into
mainfrom
gandhya-canary-errormessage

Conversation

@dgandhi62

@dgandhi62 dgandhi62 commented Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

When a command run through shell() exits with code 127, report it as a missing/unavailable executable instead of a bare exited with error code 127.

  • This change contains a major version upgrade for a dependency and I confirm all breaking changes are addressed
    • Release notes for the new version:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.64%. Comparing base (00fb6f6) to head (ca49d9d).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1998      +/-   ##
==========================================
- Coverage   91.67%   91.64%   -0.03%     
==========================================
  Files          82       82              
  Lines       13039    13039              
  Branches     1825     1824       -1     
==========================================
- Hits        11953    11950       -3     
- Misses       1048     1051       +3     
  Partials       38       38              
Flag Coverage Δ
suite.unit 91.64% <ø> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aws-cdk-automation
aws-cdk-automation added this pull request to the merge queue Sep 28, 2026
Merged via the queue into main with commit e388823 Sep 28, 2026
56 checks passed

This branch was successfully deployed

3 active (1 outdated) deployments
run-tests — ca49d9d0 Deployed Sep 25, 2026 by dgandhi62 via integ_cli (cli-integ-tests, 24.19, 1) #7003
no-approval — ca49d9d0 Deployed Sep 25, 2026 by dgandhi62 via prepare #7003
automation — 426ecd7f Deployed Sep 25, 2026 by dgandhi62 via Set AutoQueue on PR #1998 #3623
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants