-
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
Add add command #89
Add add command #89
Conversation
Add AddCommand classes in package seedu.address.logic.wanderlustlogic.wanderlustparser Update WanderlustAddCommandParser Update ParserUtil class in package seedu.address.logic.wanderlustlogic.wanderlustparser
Add AddCommand classes in package seedu.address.logic.wanderlustlogic.wanderlustparser Update WanderlustAddCommandParser Update ParserUtil class in package seedu.address.logic.wanderlustlogic.wanderlustparser
Add AddCommand classes in package seedu.address.logic.wanderlustlogic.wanderlustparser Update WanderlustAddCommandParser Update ParserUtil class in package seedu.address.logic.wanderlustlogic.wanderlustparser
Codecov Report
@@ Coverage Diff @@
## master #89 +/- ##
============================================
- Coverage 44.73% 42.94% -1.80%
Complexity 602 602
============================================
Files 148 153 +5
Lines 2754 2869 +115
Branches 332 351 +19
============================================
Hits 1232 1232
- Misses 1453 1568 +115
Partials 69 69
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
Rename Phone class to Mobile in package seedu.address.model.friend
Add AddCommand classes in package seedu.address.logic.wanderlustlogic.wanderlustparser
Update WanderlustAddCommandParser
Update ParserUtil class in package seedu.address.logic.wanderlustlogic.wanderlustparser