Skip to content

fix: update version includes a spurious newline#1477

Merged
aws-cdk-automation merged 1 commit intomainfrom
huijbers/newline
May 5, 2026
Merged

fix: update version includes a spurious newline#1477
aws-cdk-automation merged 1 commit intomainfrom
huijbers/newline

Conversation

@rix0rrr
Copy link
Copy Markdown
Contributor

@rix0rrr rix0rrr commented May 5, 2026

Our upgrade message has looked like this for a long time:

*****************************************************
*** Newer version of CDK is available [2.1120.0
] ***
*** Upgrade recommended (npm install -g aws-cdk)  ***
*****************************************************

Time to fix it and get rid of that spurious newline.


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

Our upgrade message has looked like this for a long time:

```
*****************************************************
*** Newer version of CDK is available [2.1120.0
] ***
*** Upgrade recommended (npm install -g aws-cdk)  ***
*****************************************************
```

Time to fix it and get rid of that spurious newline.
@rix0rrr rix0rrr requested a review from a team May 5, 2026 15:10
@github-actions github-actions Bot added the p2 label May 5, 2026
@aws-cdk-automation aws-cdk-automation requested a review from a team May 5, 2026 15:11
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.10%. Comparing base (acefbf8) to head (b4f002e).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1477   +/-   ##
=======================================
  Coverage   88.10%   88.10%           
=======================================
  Files          74       74           
  Lines       10669    10669           
  Branches     1457     1457           
=======================================
  Hits         9400     9400           
  Misses       1241     1241           
  Partials       28       28           
Flag Coverage Δ
suite.unit 88.10% <100.00%> (ø)

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.

@aws-cdk-automation aws-cdk-automation added this pull request to the merge queue May 5, 2026
Merged via the queue into main with commit 1f09294 May 5, 2026
45 checks passed
@aws-cdk-automation aws-cdk-automation deleted the huijbers/newline branch May 5, 2026 15:33
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.

5 participants