search

AY2324S1-CS2103T-T08-1 / tp

MIT License
0 stars 5 forks source link

Refactoring: Remove unused code #121

Closed yanghengtang closed 1 year ago

yanghengtang commented 1 year ago

Closes #119

codecov[bot] commented 1 year ago

Codecov Report

Merging #121 (3005fc9) into master (13f1d5a) will decrease coverage by 4.54%. The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #121      +/-   ##
============================================
- Coverage     83.44%   78.90%   -4.54%     
+ Complexity      855      638     -217     
============================================
  Files           122       96      -26     
  Lines          2465     1906     -559     
  Branches        265      195      -70     
============================================
- Hits           2057     1504     -553     
- Misses          369      370       +1     
+ Partials         39       32       -7
Files Coverage Δ
src/main/java/seedu/address/logic/Messages.java 92.59% <ø> (-2.15%) :arrow_down:
...main/java/seedu/address/logic/NewLogicManager.java 70.83% <100.00%> (ø)
...java/seedu/address/logic/commands/HelpCommand.java 100.00% <100.00%> (ø)
...edu/address/logic/parser/NewAddressBookParser.java 100.00% <100.00%> (ø)
...in/java/seedu/address/logic/parser/ParserUtil.java 97.36% <ø> (-0.55%) :arrow_down:
...main/java/seedu/address/model/NewModelManager.java 87.05% <100.00%> (ø)

... and 6 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more