Skip to content

Commit

Permalink
fix(cli): typescript init templates fail with error in build step (#2…
Browse files Browse the repository at this point in the history
…3130)

Fixes the compile error when running `cdk init app --language typescript && npm run build`.

Fixes: #23126 for v1

----

### All Submissions:

* [x] Have you followed the guidelines in our [Contributing guide?](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md)

### Adding new Unconventional Dependencies:

* [ ] This PR adds new unconventional dependencies following the process described [here](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md/#adding-new-unconventional-dependencies)

### New Features

* [ ] Have you added the new feature to an [integration test](https://github.com/aws/aws-cdk/blob/main/INTEGRATION_TESTS.md)?
	* [ ] Did you use `yarn integ` to deploy the infrastructure and generate the snapshot (i.e. `yarn integ` without `--dry-run`)?

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
  • Loading branch information
mrgrain committed Nov 29, 2022
1 parent d486b9a commit b06cd20
Show file tree
Hide file tree
Showing 7 changed files with 7 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/yarn-upgrade.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ jobs:
lerna exec --parallel ncu -- --upgrade --reject='@types/node,@types/prettier,@types/fs-extra,constructs,typescript,aws-sdk,aws-sdk-mock,${{ steps.list-packages.outputs.list }}' --target=minor
# Upgrade package.jsons in init templates
for pj in $(find packages/aws-cdk/lib/init-templates -name package.json); do
(cd $(dirname $pj) && ncu --upgrade --reject='@types/node,@types/prettier,@types/fs-extra,constructs,typescript,aws-sdk,aws-sdk-mock,ts-jest,jest,${{ steps.list-packages.outputs.list }}')
(cd $(dirname $pj) && ncu --upgrade --reject='@types/babel__traverse,@types/node,@types/prettier,@types/fs-extra,constructs,typescript,aws-sdk,aws-sdk-mock,ts-jest,jest,${{ steps.list-packages.outputs.list }}')
done
# Upgrade dependencies at an aws-eks integ test docker image
cd packages/@aws-cdk/aws-eks/test/sdk-call-integ-test-docker-app/app/ && ncu --upgrade --reject='@types/jest,@types/node,@types/prettier,@types/fs-extra,constructs,typescript,aws-sdk,aws-sdk-mock,ts-jest,jest,${{ steps.list-packages.outputs.list }}'
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
},
"devDependencies": {
"@aws-cdk/assertions": "%cdk-version%",
"@types/babel__traverse": "7.18.2",
"@types/prettier": "2.6.0",
"@types/jest": "^27.5.0",
"@types/node": "10.17.27",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
},
"devDependencies": {
"@aws-cdk/assertions": "%cdk-version%",
"@types/babel__traverse": "7.18.2",
"@types/jest": "^27.5.0",
"@types/prettier": "2.6.0",
"@types/node": "10.17.27",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
"devDependencies": {
"aws-cdk": "%cdk-version%",
"@aws-cdk/assertions": "%cdk-version%",
"@types/babel__traverse": "7.18.2",
"@types/jest": "^27.5.0",
"@types/node": "10.17.27",
"@types/prettier": "2.6.0",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
"cdk": "cdk"
},
"devDependencies": {
"@types/babel__traverse": "7.18.2",
"@types/jest": "^27.5.0",
"@types/node": "10.17.27",
"@types/prettier": "2.6.0",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
"test": "jest"
},
"devDependencies": {
"@types/babel__traverse": "7.18.2",
"@types/jest": "^27.5.0",
"@types/node": "10.17.27",
"@types/prettier": "2.6.0",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
"cdk": "cdk"
},
"devDependencies": {
"@types/babel__traverse": "7.18.2",
"aws-cdk": "%cdk-version%",
"@types/jest": "^27.5.0",
"@types/node": "10.17.27",
Expand Down

0 comments on commit b06cd20

Please sign in to comment.