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

Edit DG #128

Conversation

muhdharun
Copy link

Added implementation for find and check commands in the developer guide

@muhdharun muhdharun added the priority.high Must do label Oct 18, 2018
@muhdharun muhdharun added this to the v1.3 milestone Oct 18, 2018
@muhdharun muhdharun self-assigned this Oct 18, 2018
@codecov-io
Copy link

codecov-io commented Oct 18, 2018

Codecov Report

Merging #128 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #128   +/-   ##
=========================================
  Coverage     32.78%   32.78%           
  Complexity      207      207           
=========================================
  Files            61       61           
  Lines          1595     1595           
  Branches        174      174           
=========================================
  Hits            523      523           
  Misses         1044     1044           
  Partials         28       28
Impacted Files Coverage Δ Complexity Δ
src/seedu/addressbook/commands/HelpCommand.java 16.66% <ø> (ø) 1 <0> (ø) ⬇️

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 50051a0...782f89f. Read the comment docs.

Copy link

@iamputradanish iamputradanish left a comment

Choose a reason for hiding this comment

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

Good addition. Do maintain format with other group members.

@iamputradanish iamputradanish merged commit a005f5b into CS2113-AY1819S1-F10-3:master Oct 18, 2018
@iamputradanish iamputradanish changed the title Edited DG Edit DG Oct 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants