-
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
Delete ab3 code #136
Delete ab3 code #136
Conversation
# Conflicts: # src/main/java/seedu/address/logic/wanderlustlogic/wanderlustparser/CliSyntax.java
# Conflicts: # src/main/java/seedu/address/logic/wanderlustlogic/Logic.java # src/main/java/seedu/address/logic/wanderlustlogic/wanderlustparser/WanderlustGoToCommandParser.java # src/main/java/seedu/address/logic/wanderlustlogic/wanderlustparser/WanderlustParser.java # src/test/java/seedu/address/logic/parser/DeleteCommandParserTest.java # src/test/java/seedu/address/logic/wanderlustlogic/wanderlustcommands/GoToCommandTest.java
@@ -94,6 +94,7 @@ public Directory getDirectory() { | |||
return model.getDirectory(); | |||
} | |||
|
|||
// TODO: |
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.
Just clarifying here, what's there TODO here?
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.
oh cos this method is I cincai add one last time. No JavaDoc all that, need to redo
Codecov Report
@@ Coverage Diff @@
## master #136 +/- ##
============================================
- Coverage 70.76% 67.26% -3.51%
+ Complexity 1061 764 -297
============================================
Files 164 116 -48
Lines 3270 2456 -814
Branches 410 302 -108
============================================
- Hits 2314 1652 -662
+ Misses 820 681 -139
+ Partials 136 123 -13
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.
But still have some issue