Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Upgrade dependencies for all #3789

Merged
merged 2 commits into from
Jan 6, 2025
Merged

Conversation

team-tf-cdk
Copy link
Collaborator

Ran npm-check-updates and yarn upgrade to keep the yarn.lock file up-to-date.
This PR touches the following packages:

cdktf-cli\n -@cdktf/cli-core\n -cdktf\n -@cdktf/hcl2cdk\n -@cdktf/hcl2json\n -@cdktf/provider-schema\n -@cdktf/provider-generator\n -@cdktf/commons

@ansgarm
Copy link
Member

ansgarm commented Jan 6, 2025

It seems like the new types for node fail the build as we pass string | undefined to JSON.parse which only allows string now

correction: it is json-stable-stringify which now returns string | undefined

team-tf-cdk and others added 2 commits January 6, 2025 11:01
Ran npm-check-updates and yarn upgrade to keep the `yarn.lock` file up-to-date.
… returns string|undefined which is incompatible with some places we use it
@ansgarm ansgarm merged commit 22a4dc8 into main Jan 6, 2025
431 checks passed
@ansgarm ansgarm deleted the automation/yarn-upgrade-all branch January 6, 2025 10:44
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.

2 participants