Skip to content

fix(cli): error message when early validation failures cannot be retrieved is unclear#995

Merged
mrgrain merged 3 commits intoaws:mainfrom
gshpychka:patch-1
Dec 17, 2025
Merged

fix(cli): error message when early validation failures cannot be retrieved is unclear#995
mrgrain merged 3 commits intoaws:mainfrom
gshpychka:patch-1

Conversation

@gshpychka
Copy link
Contributor

@gshpychka gshpychka commented Dec 15, 2025

Clarified error message for early validation failures in deployments.

The current error message is confusing: it can be taken to mean that the CLI version is outdated, which is not the case.

It also mentions the CDK Toolkit stack version without any explanation - it is not clear which version it is referring to.

I also believe that saying "with the latest version of the CLI" is redundant - the error message will only appear in a sufficiently up-to-date CLI.


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

Clarified error message for early validation failures in deployments.
Fix error message for deployment validation failures.
auto-merge was automatically disabled December 17, 2025 07:46

Head branch was pushed to by a user without write access

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.58%. Comparing base (8ac049e) to head (37f128a).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #995   +/-   ##
=======================================
  Coverage   87.58%   87.58%           
=======================================
  Files          72       72           
  Lines       10023    10023           
  Branches     1316     1317    +1     
=======================================
  Hits         8779     8779           
  Misses       1220     1220           
  Partials       24       24           
Flag Coverage Δ
suite.unit 87.58% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 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.

@mrgrain mrgrain changed the title chore(docs): Improve error message for early validation failures fix(cli): error message for early validation failures is unclear Dec 17, 2025
@mrgrain mrgrain changed the title fix(cli): error message for early validation failures is unclear fix(cli): error message when early validation failures cannot be retrieved is unclear Dec 17, 2025
@mrgrain mrgrain added this pull request to the merge queue Dec 17, 2025
Merged via the queue into aws:main with commit 48bad5d Dec 17, 2025
41 checks passed
@gshpychka gshpychka deleted the patch-1 branch December 18, 2025 11:43
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.

3 participants