Skip to content

Commit dee84c1

Browse files
rix0rrrmrgrain
andauthored
chore: skip jsii packaging on regular build (#81)
jsii packaging is done as part of the 'release' workflow, so don't run the packaging steps as part of the normal build. This saves ~17% of build time on my machine. --- By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license Co-authored-by: Momo Kornher <[email protected]>
1 parent 2bea2b9 commit dee84c1

File tree

4 files changed

+6
-9
lines changed

4 files changed

+6
-9
lines changed

Diff for: packages/@aws-cdk/cli-lib-alpha/.projen/tasks.json

-4
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Diff for: packages/@aws-cdk/cloud-assembly-schema/.projen/tasks.json

-4
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Diff for: packages/aws-cdk/test/cli/cdk-toolkit.test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -1765,7 +1765,7 @@ class FakeCloudFormation extends Deployments {
17651765
}
17661766

17671767
function cliTest(name: string, handler: (dir: string) => void | Promise<any>): void {
1768-
test(name, () => withTempDir(handler));
1768+
test(name, () => withTempDir(handler), 120000);
17691769
}
17701770

17711771
async function withTempDir(cb: (dir: string) => void | Promise<any>) {

Diff for: projenrc/jsii.ts

+5
Original file line numberDiff line numberDiff line change
@@ -215,10 +215,15 @@ export class JsiiBuild extends pj.Component {
215215
// Only run in CI
216216
condition: 'node -e "if (!process.env.CI) process.exit(1)"',
217217
});
218+
219+
// Do not spawn 'package-all' automatically as part of 'package', the jsii packaging will
220+
// be done as part of the release task.
221+
/*
218222
tsProject.packageTask.spawn(this.packageAllTask, {
219223
// Don't run in CI
220224
condition: 'node -e "if (process.env.CI) process.exit(1)"',
221225
});
226+
*/
222227

223228
const targets: Record<string, any> = {};
224229

0 commit comments

Comments
 (0)