Skip to content

chore: remove CLI code#33474

Merged
mergify[bot] merged 16 commits intomainfrom
huijbers/move-code
Feb 18, 2025
Merged

chore: remove CLI code#33474
mergify[bot] merged 16 commits intomainfrom
huijbers/move-code

Conversation

@rix0rrr
Copy link
Copy Markdown
Contributor

@rix0rrr rix0rrr commented Feb 17, 2025

This removes the code of the CLI and CLI-related packages. Those are being moved to a different repository, as announced in #32775

The PR that adds them is here: aws/aws-cdk-cli#37


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

@rix0rrr rix0rrr requested a review from a team as a code owner February 17, 2025 09:34
@github-actions github-actions bot added the p2 label Feb 17, 2025
@aws-cdk-automation aws-cdk-automation requested a review from a team February 17, 2025 09:34
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Feb 17, 2025
@rix0rrr rix0rrr added pr-linter/cli-integ-tested Assert that any CLI changes have been integ tested pr/do-not-merge This PR should not be merged at this time. labels Feb 17, 2025
Copy link
Copy Markdown
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(This review is outdated)

@rix0rrr rix0rrr added the pr-linter/exempt-size-check The PR linter will not check the size of the PR label Feb 17, 2025
@aws-cdk-automation aws-cdk-automation dismissed their stale review February 17, 2025 09:51

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

aws-cdk-automation pushed a commit to aws/aws-cdk-cli that referenced this pull request Feb 17, 2025
This adds code from the original repositories, moving it here as
announced in aws/aws-cdk#32775.

The PR that removes the original code:
aws/aws-cdk#33474
@aws-cdk-automation aws-cdk-automation added the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Feb 17, 2025
@aws-cdk-automation aws-cdk-automation removed the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Feb 17, 2025
@rix0rrr rix0rrr removed the pr/do-not-merge This PR should not be merged at this time. label Feb 18, 2025
@aws-cdk-automation aws-cdk-automation added the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Feb 18, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.16%. Comparing base (411dc5a) to head (20e052d).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #33474      +/-   ##
==========================================
+ Coverage   81.00%   82.16%   +1.16%     
==========================================
  Files         238      119     -119     
  Lines       14271     6857    -7414     
  Branches     2492     1157    -1335     
==========================================
- Hits        11560     5634    -5926     
+ Misses       2425     1120    -1305     
+ Partials      286      103     -183     
Flag Coverage Δ
suite.unit 82.16% <ø> (+1.16%) ⬆️

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

Components Coverage Δ
packages/aws-cdk ∅ <ø> (∅)
packages/aws-cdk-lib/core 82.16% <ø> (ø)

@aws-cdk-automation
Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
  • Commit ID: 20e052d
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Feb 18, 2025

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit 8b87a63 into main Feb 18, 2025
21 checks passed
@mergify mergify bot deleted the huijbers/move-code branch February 18, 2025 13:11
@github-actions
Copy link
Copy Markdown
Contributor

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 18, 2025
@aws-cdk-automation aws-cdk-automation removed the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Feb 18, 2025
@rix0rrr rix0rrr changed the title chore: remove code chore: remove CLI code Feb 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

contribution/core This is a PR that came from AWS. p2 pr-linter/cli-integ-tested Assert that any CLI changes have been integ tested pr-linter/exempt-size-check The PR linter will not check the size of the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants