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

Created addCommentCommand.java #96

Merged
merged 3 commits into from
Apr 4, 2024

Conversation

ananya21
Copy link

@ananya21 ananya21 commented Apr 3, 2024

Created a third section within project description in which comments can be added in the UI, Created AddCommentCommand.java, AddCommentCommandParser.java, and edited AddressBookParser.java

Resolves issue#85

…ion in which comments can be added in the UI, Created AddCommentCommand.java, AddCommentCommandParser.java, and edited AddressBookParser.java, resolves issue#85
@ananya21 ananya21 requested a review from modembcc April 3, 2024 19:11
Copy link

codecov bot commented Apr 3, 2024

Codecov Report

Attention: Patch coverage is 1.47059% with 67 lines in your changes are missing coverage. Please review.

Project coverage is 55.08%. Comparing base (a0b2d57) to head (bd39e30).

❗ Current head bd39e30 differs from pull request most recent head 6509297. Consider uploading reports for the commit 6509297 to get more accurate results

Files Patch % Lines
...eedu/address/logic/commands/AddCommentCommand.java 0.00% 25 Missing ⚠️
.../address/logic/parser/AddCommentCommandParser.java 0.00% 17 Missing ⚠️
src/main/java/seedu/address/ui/TaskListPanel.java 0.00% 10 Missing ⚠️
...main/java/seedu/address/model/project/Comment.java 0.00% 6 Missing ⚠️
src/main/java/seedu/address/ui/CommentCard.java 0.00% 5 Missing ⚠️
...c/main/java/seedu/address/model/person/Person.java 25.00% 3 Missing ⚠️
.../seedu/address/logic/parser/AddressBookParser.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #96      +/-   ##
============================================
+ Coverage     54.76%   55.08%   +0.32%     
  Complexity      453      453              
============================================
  Files           105      105              
  Lines          2014     2004      -10     
  Branches        217      213       -4     
============================================
+ Hits           1103     1104       +1     
+ Misses          857      846      -11     
  Partials         54       54              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

@Lin-Shuang-Shuang Lin-Shuang-Shuang left a comment

Choose a reason for hiding this comment

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

LGTM

@ananya21 ananya21 merged commit c74afda into AY2324S2-CS2103T-W08-4:master Apr 4, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants