-
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
Modify UG #220
Modify UG #220
Conversation
Codecov Report
@@ Coverage Diff @@
## master #220 +/- ##
============================================
+ Coverage 72.28% 75.37% +3.08%
- Complexity 725 756 +31
============================================
Files 112 112
Lines 2555 2530 -25
Branches 275 272 -3
============================================
+ Hits 1847 1907 +60
+ Misses 601 530 -71
+ Partials 107 93 -14
... and 11 files with indirect coverage changes 📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today! |
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! Resolved merge conflicts but will create another PR to fine tune the UG to combine both mine and Faiz's parts.
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.
Instructions at the add-doctor part seems a bit long, other than that looks good.
6ffae7b
into
AY2324S1-CS2103T-T09-3:master
Modify UG to Address Edge cases associated with
add-doctor
command.