Skip to content
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 view command for displaying calorie lists #213

Merged
merged 5 commits into from
Nov 1, 2020

Conversation

ethanso56
Copy link

Format: view INDEX

@codecov-io
Copy link

codecov-io commented Oct 31, 2020

Codecov Report

Merging #213 into master will increase coverage by 0.08%.
The diff coverage is 42.22%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #213      +/-   ##
============================================
+ Coverage     41.95%   42.04%   +0.08%     
- Complexity      517      529      +12     
============================================
  Files           115      117       +2     
  Lines          2734     2764      +30     
  Branches        331      334       +3     
============================================
+ Hits           1147     1162      +15     
- Misses         1461     1478      +17     
+ Partials        126      124       -2     
Impacted Files Coverage Δ Complexity Δ
...edu/address/logic/parser/MyFitnessBuddyParser.java 64.00% <0.00%> (-2.67%) 12.00 <0.00> (ø)
src/main/java/seedu/address/ui/DayCard.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
src/main/java/seedu/address/ui/MainWindow.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...java/seedu/address/logic/commands/ViewCommand.java 41.66% <41.66%> (ø) 5.00 <5.00> (?)
...va/seedu/address/logic/commands/CommandResult.java 58.33% <50.00%> (-3.58%) 20.00 <4.00> (+2.00) ⬇️
.../seedu/address/logic/parser/ViewCommandParser.java 100.00% <100.00%> (ø) 2.00 <2.00> (?)
...va/seedu/address/logic/commands/DeleteCommand.java 61.53% <0.00%> (+15.38%) 6.00% <0.00%> (+3.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 74e6bad...b3bc7ae. Read the comment docs.

@junhui-phoon junhui-phoon self-requested a review October 31, 2020 18:52
Copy link

@junhui-phoon junhui-phoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just need to include the javadocs

@junhui-phoon junhui-phoon linked an issue Oct 31, 2020 that may be closed by this pull request
@ethanso56 ethanso56 merged commit 2859136 into AY2021S1-CS2103T-W11-3:master Nov 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[PE-D] Need for mouse clicking
3 participants