chore: fix build with new jsii version#18850
Conversation
Fix some issues that break the build with either the upcoming jsii release, or a local jsii build.
|
|
| "noUnusedLocals": true, | ||
| "noUnusedParameters": true, | ||
| "noImplicitReturns": true, | ||
| "experimentalDecorators": true, |
There was a problem hiding this comment.
Note for posterity / other people who come across this PR and wonder:
This shouldn't be necessary unless you install jsii-reflect from a local source tree (which I assume is what you did to validate this PR) without adding a project reference in tsconfig.json (because then, tsc will read the .ts files which contain a decorator -- @memoized -- instead of the .d.ts files where the decorator is stripped).
|
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
|
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
Fix some issues that break the build with either the upcoming jsii release, or a local jsii build. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Fix some issues that break the build with either the upcoming jsii
release, or a local jsii build.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license