Skip to content

Commit

Permalink
chore(toolkit): stricter config
Browse files Browse the repository at this point in the history
  • Loading branch information
mrgrain committed Jan 15, 2025
1 parent 312bb94 commit 7cd792c
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 3 deletions.
1 change: 1 addition & 0 deletions packages/@aws-cdk/toolkit/.npmignore
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ coverage
tsconfig.json
*.tsbuildinfo
junit.xml
jest.config.js

# Include .jsii
!.jsii
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import * as fs from 'fs-extra';
import { lte } from 'semver';
import type { AppSynthOptions } from './source-builder';
import { ToolkitError } from '../../errors';
import { ActionAwareIoHost, asLogger, error, warn } from '../../io/private';
import { ActionAwareIoHost, asLogger, error } from '../../io/private';
import { ToolkitServices } from '../../toolkit/private';

export { guessExecutable } from 'aws-cdk/lib/api/cxapp/exec';
Expand Down
5 changes: 3 additions & 2 deletions packages/@aws-cdk/toolkit/lib/toolkit.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ import { HotswapMode } from 'aws-cdk/lib/api/hotswap/common';
import { StackActivityProgress } from 'aws-cdk/lib/api/util/cloudformation/stack-activity-monitor';
import { ResourceMigrator } from 'aws-cdk/lib/migrator';
import { obscureTemplate, serializeStructure } from 'aws-cdk/lib/serialize';
import { Configuration } from 'aws-cdk/lib/settings';
import { tagsForStack } from 'aws-cdk/lib/tags';
import { CliIoHost } from 'aws-cdk/lib/toolkit/cli-io-host';
import { validateSnsTopicArn } from 'aws-cdk/lib/util/validate-notification-arn';
Expand Down Expand Up @@ -160,6 +159,8 @@ export class Toolkit extends CloudAssemblySourceBuilder {
public async list(cx: ICloudAssemblySource, _options: ListOptions): Promise<void> {
const ioHost = withAction(this.ioHost, 'list');
const assembly = await this.assemblyFromSource(cx);
ioHost;
assembly;
throw new Error('Not implemented yet');
}

Expand Down Expand Up @@ -201,7 +202,7 @@ export class Toolkit extends CloudAssemblySourceBuilder {
});
await migrator.tryMigrateResources(stackCollection, options);

const requireApproval = options.requireApproval ?? RequireApproval.BROADENING;
// const requireApproval = options.requireApproval ?? RequireApproval.BROADENING;

const parameterMap = buildParameterMap(options.parameters?.parameters);

Expand Down
2 changes: 2 additions & 0 deletions packages/@aws-cdk/toolkit/tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,8 @@
"strict": true,
"noUncheckedIndexedAccess": true,
"noImplicitOverride": true,
"noUnusedLocals": true,
"noUnusedParameters": true,
"module": "NodeNext",
"declaration": true,
"declarationMap": true,
Expand Down

0 comments on commit 7cd792c

Please sign in to comment.