search

AY2122S2-CS2103T-T09-3 / tp

MIT License
0 stars 4 forks source link

Update code quality of all commons and ui classes/methods #299

Closed JonathanHoshi closed 2 years ago

JonathanHoshi commented 2 years ago

Added header comments of all public classes and methods of commons and ui folders in both main and test.

JonathanHoshi commented 2 years ago

iinw, you have to start the first sentence with a verb. image

Is it okay for you give me an example of how to write the header for public test methods?

codecov-commenter commented 2 years ago

Codecov Report

Merging #299 (3b1c52f) into master (01b62e1) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #299   +/-   ##
=========================================
  Coverage     62.28%   62.28%           
  Complexity      480      480           
=========================================
  Files            88       88           
  Lines          1710     1710           
  Branches        194      194           
=========================================
  Hits           1065     1065           
  Misses          561      561           
  Partials         84       84
Impacted Files Coverage Δ
...ava/seedu/trackermon/commons/core/GuiSettings.java 80.76% <ø> (ø)
...n/java/seedu/trackermon/commons/core/JarTools.java 40.62% <ø> (ø)
...java/seedu/trackermon/commons/core/LogsCenter.java 75.00% <ø> (ø)
...in/java/seedu/trackermon/commons/core/Version.java 82.50% <ø> (ø)
...ava/seedu/trackermon/commons/core/index/Index.java 100.00% <ø> (ø)
...on/commons/exceptions/DataConversionException.java 100.00% <ø> (ø)
...rmon/commons/exceptions/IllegalValueException.java 50.00% <ø> (ø)
...seedu/trackermon/logic/commands/CommandResult.java 88.88% <ø> (ø)
...java/seedu/trackermon/storage/JsonFileManager.java 0.00% <ø> (ø)
.../main/java/seedu/trackermon/ui/CommandBuilder.java 100.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 01b62e1...3b1c52f. Read the comment docs.