Skip to content

Commit

Permalink
feat: nested submodules, S3 retries (#1360)
Browse files Browse the repository at this point in the history
Bring in cdklabs/jsii-docgen#1215 which adds
support for nested submodules (`asm.sub1.sub2`), and adjust the code to
match.

Also add S3 throttling protection via retries and backoff, since we have
a fair bit of that going around as well.


----

*By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache-2.0 license*
  • Loading branch information
rix0rrr authored Nov 21, 2023
1 parent d53d563 commit d1c626f
Show file tree
Hide file tree
Showing 12 changed files with 216 additions and 666 deletions.
5 changes: 1 addition & 4 deletions .projen/deps.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions .projen/tasks.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 4 additions & 1 deletion .projenrc.ts
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@ const project = new CdklabsConstructLibrary({
'fs-extra',
'got',
'JSONStream',
'jsii-reflect',
'semver',
'spdx-license-list',
'streamx',
Expand Down Expand Up @@ -198,6 +197,10 @@ project.addTask('bundle', {
description: 'Bundle all lambda and ECS functions',
});

// I have to put this here, otherwise projen overrides this with a `jsii-docgen`
// dependency without a version. I don't know where or why.
project.addDevDeps('jsii-docgen@^10.2.0');

// extract the "build/" directory from "construct-hub-webapp" into "./website"
// and bundle it with this library. this way, we are only taking a
// dev-dependency on the webapp instead of a normal/bundled dependency.
Expand Down
Loading

0 comments on commit d1c626f

Please sign in to comment.