-
Notifications
You must be signed in to change notification settings - Fork 4
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
Support task deadline #139
Support task deadline #139
Conversation
Codecov Report
@@ Coverage Diff @@
## master #139 +/- ##
============================================
- Coverage 59.42% 59.26% -0.16%
- Complexity 597 598 +1
============================================
Files 109 109
Lines 2228 2234 +6
Branches 274 275 +1
============================================
Hits 1324 1324
- Misses 825 830 +5
- Partials 79 80 +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 especially for changing the JavaFXVersion 👍
@@ -42,7 +42,7 @@ task coverage(type: JacocoReport) { | |||
|
|||
dependencies { | |||
String jUnitVersion = '5.4.0' | |||
String javaFxVersion = '11' | |||
String javaFxVersion = '11.0.1' |
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.
WOW this has been bugging me for the longest time, thanks for finding and changing it
@@ -37,8 +39,11 @@ public AddTaskCommand parse(String args) throws ParseException { | |||
ParserUtil.parseTaskBasicInformation(argMultimap.getValue(PREFIX_TASK_PROGRESS).get())); | |||
boolean taskStatus = Boolean.parseBoolean( | |||
ParserUtil.parseTaskBasicInformation(argMultimap.getValue(PREFIX_TASK_IS_DONE).get())); | |||
|
|||
Task task = new Task(taskName, null, null, taskProgress, taskStatus); | |||
Deadline taskDeadline = null; |
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.
Why set it to null here? Why not declare the variable below.
@@ -46,7 +46,7 @@ public TaskFilterCommand parse(String args) throws ParseException { | |||
|
|||
if (argMultimap.getValue(PREFIX_TASK_DEADLINE).isPresent()) { | |||
Deadline deadline = ParserUtil.parseDeadline(argMultimap.getValue(PREFIX_TASK_DEADLINE).get()); | |||
predicate = task -> task.getDeadline().equals(deadline); | |||
predicate = task -> task.isDueOn(deadline); |
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 adhering to Law of Demeter
No description provided.