search

AY2122S2-CS2103T-T17-1 / tp

About Tinner
https://ay2122s2-cs2103t-t17-1.github.io/tp/
MIT License
1 stars 5 forks source link

Update execution of commandResult #295

Closed BrendonLau closed 2 years ago

BrendonLau commented 2 years ago

Update the correct execution of commandResult for every feature implementation.

e.g.

  1. The Parser returns the FavouriteCompanyCommand to LogicManager which then returns the CommandResult to MainWindow in order to display the changes to the user.
codecov-commenter commented 2 years ago

Codecov Report

Merging #295 (f243ad5) into master (3b0fd93) will decrease coverage by 0.03%. The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #295      +/-   ##
============================================
- Coverage     68.89%   68.85%   -0.04%     
  Complexity      709      709              
============================================
  Files           107      107              
  Lines          2218     2219       +1     
  Branches        263      263              
============================================
  Hits           1528     1528              
- Misses          605      606       +1     
  Partials         85       85
Impacted Files Coverage Δ
src/main/java/seedu/tinner/ui/CompanyCard.java 0.00% <0.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 3b0fd93...f243ad5. Read the comment docs.