Skip to content

Commit e629e30

Browse files
kaizenccmrgrain
andauthored
chore(cli): type currentAction as CliAction (#571)
`CliAction` type feels slightly off as it should be `CliAction` and not `ToolkitAction`. This makes the default make sense. Either way, the internal property of `CliIoHost` is typed to `CliAction`. In practice this probably doesn't matter because I don't think we are initializing `CliIoHost` with anything but `none`. --- 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 <kornherm@amazon.co.uk>
1 parent 666c9cc commit e629e30

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

packages/@aws-cdk/toolkit-lib/lib/api/io/toolkit-action.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/**
2-
* The current action being performed by the CLI. 'none' represents the absence of an action.
2+
* The current action being performed by the Toolkit.
33
*/
44
export type ToolkitAction =
55
| 'assembly'

packages/aws-cdk/lib/cli/io-host/cli-io-host.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,9 @@ import { StackActivityProgress } from '../../commands/deploy';
1010

1111
export type { IIoHost, IoMessage, IoMessageCode, IoMessageLevel, IoRequest };
1212

13+
/**
14+
* The current action being performed by the CLI. 'none' represents the absence of an action.
15+
*/
1316
type CliAction =
1417
| ToolkitAction
1518
| 'context'
@@ -24,7 +27,7 @@ export interface CliIoHostProps {
2427
*
2528
* @default 'none'
2629
*/
27-
readonly currentAction?: ToolkitAction;
30+
readonly currentAction?: CliAction;
2831

2932
/**
3033
* Determines the verbosity of the output.

0 commit comments

Comments
 (0)