cdklabs / cdklabs/cdk-from-cfn

cdk migrate cli UX issue

Open
#367 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
71
Forks
21
Avg merge
4h 6m
Merged PRs (30d)
17

Description

When not supplying stack name an error with unclear messaging is shown. Expectation state the missing parameters

```
$ ./node_modules/cdk/bin/cdk migrate --from-path template.yaml --output-path cdkstack/
❌ Migrate failed for undefined: stack generation failed due to error
'The text and separator arguments should be of type string'
```

supplying --stack-name test as well resolves the issue but it was unclear that this was needed from the error.

Contributor guide

Open the contributing guide

Research direction

Reproduce the issue with the shown cdk migrate command, first without --stack-name and then with it. Inspect the migrate CLI handling of the required stack name; done means the missing parameter is identified clearly instead of producing the current undefined and type error message.

Written by the indexing model from the issue text.

Assessment

Tech stack
aws, rust
Domain
cli, cloud
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.