-
Notifications
You must be signed in to change notification settings - Fork 42
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(deps): bumping cdk from 1.116.0 to 1.129.0 #618
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like there's a whole pile of additional dependencies to sort through -- they added a bunch in the last 12 versions of CDK, or something, it seems.
From the CI:
aws-rfdk: $ cdk-build
aws-rfdk: warning JSII6: A "peerDependency" on "@aws-cdk/assets" at "1.128.0" means you should take a "devDependency" on "@aws-cdk/assets" at "1.128.0" (found "undefined")"
aws-rfdk: warning JSII6: A "peerDependency" on "@aws-cdk/aws-apigateway" at "1.128.0" means you should take a "devDependency" on "@aws-cdk/aws-apigateway" at "1.128.0" (found "undefined")"
aws-rfdk: warning JSII6: A "peerDependency" on "@aws-cdk/aws-apigatewayv2" at "1.128.0" means you should take a "devDependency" on "@aws-cdk/aws-apigatewayv2" at "1.128.0" (found "undefined")"
... and so on
examples/deadline/All-In-AWS-Infrastructure-Basic/ts/bin/app.ts
Outdated
Show resolved
Hide resolved
examples/deadline/All-In-AWS-Infrastructure-Basic/ts/bin/app.ts
Outdated
Show resolved
Hide resolved
examples/deadline/All-In-AWS-Infrastructure-Basic/ts/bin/config.ts
Outdated
Show resolved
Hide resolved
examples/deadline/All-In-AWS-Infrastructure-Basic/ts/lib/service-tier.ts
Outdated
Show resolved
Hide resolved
19b4cec
to
4746cea
Compare
I've removed my test code but am still looking into why all those |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than the feedback already brought up by @ddneilson, this looks good. Two small things with code suggestions here:
b5645e8
to
3f8c8de
Compare
3f8c8de
to
d351ddf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like there's still one more CI issue on the yarn build
step:
aws-rfdk: $ cdk-build
aws-rfdk: warning JSII6: A "peerDependency" on "constructs" at "^3.3.69" means you should take a "devDependency" on "constructs" at "3.3.69" (found "^3.3.69")"
Discussed off-line. We'll have to accept this warning. |
Upgraded all our dependencies to prep for our next RFDK release.
Testing was done by running the build in the base directory along with all the typescript example directories, and deploying the basic example. I also upgraded JSII in our pipeline and deployed a test pipeline that ran against these changes.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license