Skip to content

Conversation

@aws-cdk-automation
Copy link
Collaborator

Ran npm-check-updates and yarn upgrade to keep the yarn.lock file up-to-date.

@aws-cdk-automation aws-cdk-automation added auto-approve contribution/core This is a PR that came from AWS. dependencies This issue is a problem in a dependency or a pull request that updates a dependency file. labels Jul 2, 2025
@aws-cdk-automation aws-cdk-automation requested a review from a team July 2, 2025 13:49
@aws-cdk-automation aws-cdk-automation requested a review from a team July 2, 2025 13:49
@github-actions github-actions bot added the p2 label Jul 2, 2025
@aws-cdk-automation aws-cdk-automation force-pushed the automation/yarn-upgrade branch from 993a57a to 4a0eba1 Compare July 8, 2025 08:54
@leonmk-aws leonmk-aws self-assigned this Jul 8, 2025
@leonmk-aws
Copy link
Contributor

Reran the workflow because there were conflicts: https://github.com/aws/aws-cdk/actions/runs/16138538595

@leonmk-aws leonmk-aws force-pushed the automation/yarn-upgrade branch from 4a0eba1 to 4523498 Compare July 9, 2025 11:20
@aws-cdk-automation aws-cdk-automation force-pushed the automation/yarn-upgrade branch 2 times, most recently from 06ff40a to 6c4bef9 Compare July 10, 2025 12:17
@leonmk-aws leonmk-aws added pr/do-not-merge This PR should not be merged at this time. and removed pr/do-not-merge This PR should not be merged at this time. labels Jul 10, 2025
@aws-cdk-automation aws-cdk-automation force-pushed the automation/yarn-upgrade branch 2 times, most recently from 50242c2 to 2d2fafc Compare July 23, 2025 13:52
@mergify
Copy link
Contributor

mergify bot commented Jul 23, 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 bot pushed a commit that referenced this pull request Jul 29, 2025
### Issue # (if applicable)

Related to the #34888 PR.

This PR contains the commit created by the Github action (that doesn't build) and another commit with the fix (making it easier to review).

### Reason for this change

Version 6.0.0 of `@types/minimatch` got released on [NPM](https://www.npmjs.com/package/@types/minimatch) which deprecates the package and now the package doesn't contain code anymore (see files on NPM).

We cannot upgrade our version of `glob` as this requires Node.js 20, see #33231

### Description of changes
Pin `@types/minimatch` to the previous working version.

This has a side effect: lerna uses a pinned version of @types/minimatch (you can verify this with `yarn why @types/minimatch` with version `3.x.x`, now it will use `5.1.2`. As the build still passes I think this doesn't have consequences.

I also removed `@types/minimatch` from the `nohoist` as this line doesn't have any effect since all the packages in the repo now use the same version.


### Description of how you validated changes

Build, integ and unit tests. Ran `yarn why @types/minimatch` to confirm that all the packages use the same version of minimatch.

### Checklist
- [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md)

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
@aws-cdk-automation aws-cdk-automation force-pushed the automation/yarn-upgrade branch from 2d2fafc to 4e2061d Compare July 30, 2025 11:26
@mergify
Copy link
Contributor

mergify bot commented Jul 30, 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).

@aws-cdk-automation aws-cdk-automation added the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Jul 30, 2025
@mergify
Copy link
Contributor

mergify bot commented Jul 30, 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).

@leonmk-aws leonmk-aws removed their assignment Jul 31, 2025
Ran npm-check-updates and yarn upgrade to keep the `yarn.lock` file up-to-date.
@aws-cdk-automation aws-cdk-automation force-pushed the automation/yarn-upgrade branch from bc7178f to 74d56a9 Compare July 31, 2025 14:50
@aws-cdk-automation
Copy link
Collaborator Author

AWS CodeBuild CI Report

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

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

@mergify
Copy link
Contributor

mergify bot commented Jul 31, 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 5682cbe into main Jul 31, 2025
20 checks passed
@mergify mergify bot deleted the automation/yarn-upgrade branch July 31, 2025 15:22
@github-actions
Copy link
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 Jul 31, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

auto-approve contribution/core This is a PR that came from AWS. dependencies This issue is a problem in a dependency or a pull request that updates a dependency file. p2 pr/needs-maintainer-review This PR needs a review from a Core Team Member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants