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 low stocks to update #214

Conversation

AudreyFelicio
Copy link

No description provided.

@codecov-io
Copy link

Codecov Report

Merging #214 into master will decrease coverage by 0.28%.
The diff coverage is 39.39%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #214      +/-   ##
============================================
- Coverage     53.79%   53.51%   -0.29%     
- Complexity      661      663       +2     
============================================
  Files           121      121              
  Lines          3032     3061      +29     
  Branches        406      409       +3     
============================================
+ Hits           1631     1638       +7     
- Misses         1269     1288      +19     
- Partials        132      135       +3     
Impacted Files Coverage Δ Complexity Δ
...du/stock/logic/parser/SuggestionCommandParser.java 80.47% <ø> (-0.69%) 63.00 <0.00> (-1.00)
...in/java/seedu/stock/model/stock/QuantityAdder.java 25.00% <15.38%> (-12.50%) 3.00 <0.00> (ø)
...main/java/seedu/stock/logic/parser/ParserUtil.java 73.97% <16.66%> (-5.14%) 22.00 <1.00> (+1.00) ⬇️
.../seedu/stock/logic/parser/UpdateCommandParser.java 80.39% <71.42%> (-4.61%) 15.00 <0.00> (+2.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 28946a1...85effea. Read the comment docs.

Copy link

@kaichaoang kaichaoang left a comment

Choose a reason for hiding this comment

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

LGTM!

@kaichaoang kaichaoang merged commit 3c887b3 into AY2021S1-CS2103T-T15-3:master Oct 28, 2020
@AudreyFelicio AudreyFelicio added this to the v1.3 milestone Oct 28, 2020
@AudreyFelicio AudreyFelicio self-assigned this Oct 28, 2020
@AudreyFelicio AudreyFelicio linked an issue Oct 28, 2020 that may be closed by this pull request
@AudreyFelicio AudreyFelicio added the type.Task Task that needs to be done label Oct 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type.Task Task that needs to be done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add low stocks to update
3 participants