-
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
Dg update2 #371
Dg update2 #371
Conversation
Codecov Report
@@ Coverage Diff @@
## master #371 +/- ##
=========================================
Coverage 61.59% 61.59%
Complexity 796 796
=========================================
Files 136 136
Lines 3101 3101
Branches 346 346
=========================================
Hits 1910 1910
Misses 1062 1062
Partials 129 129 Continue to review full report at Codecov.
|
Expected: Similar to previous. | ||
|
||
1. _{ more test cases … }_ |
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.
Thanks for removing these!
@@ -5,8 +5,8 @@ hide circle | |||
|
|||
class AddXYZCommand { | |||
/' Attributes '/ | |||
+ MESSAGE_SUCCESS: String | |||
+ MESSAGE_INVALID: String | |||
<u>+ MESSAGE_SUCCESS: String |
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.
Good note of static variables!
@@ -33,7 +31,7 @@ AddressBookParser -> AddCommandParser : parse("-e...") | |||
activate AddCommandParser | |||
|
|||
activate TaskCommandParser | |||
AddCommandParser -> TaskCommandParser : parse("-e...") | |||
AddCommandParser -> TaskCommandParser : parseEvent("-e...") |
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.
For Add Tasks do we not need a ParserUtil class?
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.
Ok I will add it in the next pr, thanks for mentioning it
No description provided.