Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

toolkit (ui): "cdk context" with empty context looks odd #1549

Closed
eladb opened this issue Jan 15, 2019 · 0 comments
Closed

toolkit (ui): "cdk context" with empty context looks odd #1549

eladb opened this issue Jan 15, 2019 · 0 comments
Labels
bug This issue is a bug. package/tools Related to AWS CDK Tools or CLI ui Related to CLI User Interface issues

Comments

@eladb
Copy link
Contributor

eladb commented Jan 15, 2019 

Context found in cdk.json:

┌──┬──────────────────────────────────────────────────┬──────────────────────────────────────────────────┐
│ … │ Key                                              │ Value                                            │
└──┴──────────────────────────────────────────────────┴──────────────────────────────────────────────────┘
Run cdk context --reset KEY_OR_NUMBER to remove a context key. It will be refreshed on the next CDK synthesis run.
@eladb eladb added bug This issue is a bug. package/tools Related to AWS CDK Tools or CLI ui Related to CLI User Interface issues labels Jan 15, 2019
rix0rrr added a commit that referenced this issue Jan 16, 2019
@rix0rrr
Use same table formatting in CDK toolkit as CFN-diff
963920e 
Rewrite 'undefined' logic to make it less onerous.

Print a message instead of an empty table if 'cdk context' has no
information in it (fixes #1549).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug This issue is a bug. package/tools Related to AWS CDK Tools or CLI ui Related to CLI User Interface issues
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@eladb