-
Notifications
You must be signed in to change notification settings - Fork 5
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.
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
Update DG #341
Update DG #341
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #341 +/- ##
============================================
+ Coverage 77.38% 77.70% +0.31%
- Complexity 708 713 +5
============================================
Files 103 103
Lines 2198 2202 +4
Branches 224 224
============================================
+ Hits 1701 1711 +10
+ Misses 442 439 -3
+ Partials 55 52 -3 ☔ View full report in Codecov by Sentry. |
|
||
/** | ||
* Creates a GatherCommand object for Financial Plan from {@code argString} | ||
*/ | ||
private GatherCommand createGatherByFinancialPlan(String argString) throws ParseException { | ||
String financialPlanArg = removePrefix(argString, PREFIX_FINANCIAL_PLAN); | ||
validateFinancialPlan(financialPlanArg); | ||
assert isValidFinancialPlanName(financialPlanArg) : "Prompt has to meets valid FP requirements"; | ||
GatherEmailByFinancialPlan prompt = new GatherEmailByFinancialPlan(financialPlanArg); | ||
return new GatherCommand(prompt); | ||
} | ||
|
||
/** | ||
* Creates a GatherCommand object for Tag from {@code argString} | ||
*/ | ||
private GatherCommand createGatherByTag(String argString) throws ParseException { | ||
String tagArg = removePrefix(argString, PREFIX_TAG); | ||
validateTag(tagArg); | ||
assert isValidTagName(tagArg) : "Prompt has to meets valid Tag requirements"; | ||
GatherEmailByTag prompt = new GatherEmailByTag(tagArg); | ||
return new GatherCommand(prompt); | ||
} | ||
} |
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.
added by @AlyssaPng
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
No description provided.