search

AY2324S2-CS2103T-W10-2 / tp

https://nus-cs2103-ay2324s2.github.io/tp/
MIT License
1 stars 5 forks source link

Fix code quality and issues #309

Closed Joshy837 closed 5 months ago

Joshy837 commented 5 months ago

Fix some code quality and UG issues

codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 86.86869% with 13 lines in your changes are missing coverage. Please review.

Project coverage is 77.06%. Comparing base (28f9cb4) to head (d4191b1).

:exclamation: Current head d4191b1 differs from pull request most recent head 9cb1d84. Consider uploading reports for the commit 9cb1d84 to get more accurate results

Files Patch % Lines
...in/java/seedu/address/logic/parser/ParserUtil.java 0.00% 8 Missing :warning:
...a/seedu/address/model/person/UniquePersonList.java 95.55% 2 Missing and 2 partials :warning:
.../seedu/address/logic/parser/SortCommandParser.java 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #309 +/- ## ============================================ + Coverage 76.89% 77.06% +0.16% - Complexity 902 908 +6 ============================================ Files 131 131 Lines 3216 3244 +28 Branches 372 360 -12 ============================================ + Hits 2473 2500 +27 Misses 601 601 - Partials 142 143 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.