-
Notifications
You must be signed in to change notification settings - Fork 522
chore: updates error msg in addpool cmd #4446
Conversation
Signed-off-by: Nilekh Chaudhari <1626598+nilekhc@users.noreply.github.com>
💖 Thanks for opening your first pull request! 💖 We use semantic commit messages to streamline the release process. Before your pull request can be merged, you should make sure your first commit and PR title start with a semantic prefix. Examples of commit messages with semantic prefixes: - |
/azp pr-e2e |
Command 'pr-e2e' is not supported by Azure Pipelines. Supported commands
See additional documentation. |
/azp run pr-e2e |
Azure Pipelines successfully started running 1 pipeline(s). |
Codecov Report
@@ Coverage Diff @@
## master #4446 +/- ##
========================================
Coverage 72.04% 72.05%
========================================
Files 141 141
Lines 21631 21764 +133
========================================
+ Hits 15584 15681 +97
- Misses 5096 5131 +35
- Partials 951 952 +1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
thanks @nilekhc
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jackfrancis, nilekhc The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Congrats on merging your first pull request! 🎉🎉🎉 |
Signed-off-by: Nilekh Chaudhari 1626598+nilekhc@users.noreply.github.com
Reason for Change:
This makes flag name in error message consistent with flag required by
addpool
cmdIssue Fixed:
Credit Where Due:
Does this change contain code from or inspired by another project?
If "Yes," did you notify that project's maintainers and provide attribution?
Requirements:
Notes: