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.

Sign up for GitHub

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

Jump to bottom

Edit add command example and find command example and parameters #137

Merged
merged 4 commits into from
Nov 4, 2020
Merged

Edit add command example and find command example and parameters #137

merged 4 commits into from
Nov 4, 2020

Conversation

joshruien
Copy link

No description provided.

@joshruien joshruien added the bug Something isn't working label Nov 3, 2020
@joshruien joshruien added this to the v1.4 milestone Nov 3, 2020
@joshruien joshruien self-assigned this Nov 3, 2020
@joshruien joshruien linked an issue Nov 3, 2020 that may be closed by this pull request
[PE-D] The add command prompt does not reflect the context of the current product #129
Closed
@joshruien joshruien changed the title Edit add command example Edit add command example and find command example and parameters Nov 3, 2020
@joshruien joshruien linked an issue Nov 3, 2020 that may be closed by this pull request
[PE-D] Incorrect example of command usage for delete #123
Closed
jonfoocy
jonfoocy approved these changes Nov 4, 2020
View reviewed changes
Copy link

@jonfoocy jonfoocy left a comment

Choose a reason for hiding this comment

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

LGTM!

@codecov-io
Copy link

Codecov Report

Merging #137 into master will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #137      +/-   ##
============================================
+ Coverage     69.35%   69.37%   +0.02%     
- Complexity      447      448       +1     
============================================
  Files            78       78              
  Lines          1377     1378       +1     
  Branches        147      147              
============================================
+ Hits            955      956       +1     
  Misses          376      376              
  Partials         46       46
Impacted Files Coverage Δ Complexity Δ
...va/seedu/medmoriser/logic/commands/AddCommand.java 85.71% <ø> (ø) 8.00 <0.00> (ø)
...a/seedu/medmoriser/logic/commands/FindCommand.java 90.90% <ø> (+0.43%) 10.00 <0.00> (+1.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 3931203...df51235. Read the comment docs.

@joshruien joshruien merged commit cadfc71 into AY2021S1-CS2103T-W15-1:master Nov 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jonfoocy jonfoocy jonfoocy approved these changes

@ChengJiyuqing ChengJiyuqing ChengJiyuqing approved these changes

Assignees

@joshruien joshruien

Labels
bug Something isn't working
Projects
Team Project
Awaiting triage
Milestone
v1.4
4 participants
@joshruien @codecov-io @jonfoocy @ChengJiyuqing