Skip to content
This repository was archived by the owner on Apr 13, 2020. It is now read-only.

[FEATURE] Added code to have error chaining in deployment create cmd.#477

Merged
dennisseah merged 5 commits intomasterfrom
issue1283
Mar 31, 2020
Merged

[FEATURE] Added code to have error chaining in deployment create cmd.#477
dennisseah merged 5 commits intomasterfrom
issue1283

Conversation

@dennisseah
Copy link
Collaborator

@dennisseah dennisseah commented Mar 31, 2020

closes microsoft/bedrock#1283

e.g.

$ ts-node src/index.ts deployment create
error:   
code: 1000
message: introspect-create-cmd-failed: Deployment create command was not successfully executed.
  code: 1001
  message: introspect-create-cmd-missing-values: Access key, storage account name, partition key and/or table name were not provided. Provide them.

@dennisseah dennisseah requested review from ali92hm, andrebriggs, edaena, mtarng and samiyaakhtar and removed request for ali92hm March 31, 2020 01:43
@dennisseah dennisseah merged commit 98f2f92 into master Mar 31, 2020
@dennisseah dennisseah deleted the issue1283 branch April 6, 2020 22:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@edaena edaena Awaiting requested review from edaena

@mtarng mtarng Awaiting requested review from mtarng

2 more reviewers

@samiyaakhtar samiyaakhtar samiyaakhtar approved these changes

@andrebriggs andrebriggs andrebriggs approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Have specific error codes and error chain for deployment create command

3 participants

@dennisseah @samiyaakhtar @andrebriggs