AY2122S1-CS2103T-F12-2 / tp

https://ay2122s1-cs2103t-f12-2.github.io/tp/
MIT License
1 stars 5 forks source link

Standardise APPLICATION_STATUS, refactor ab3 diagrams #273

Closed Chang-CH closed 2 years ago

Chang-CH commented 2 years ago

As raised by @Jeromesyl

Closes #274 , #275

codecov-commenter commented 2 years ago

Codecov Report

Merging #273 (55bb50b) into master (46845a4) will decrease coverage by 0.03%. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #273      +/-   ##
============================================
- Coverage     70.81%   70.78%   -0.04%     
+ Complexity      596      595       -1     
============================================
  Files            85       85              
  Lines          1919     1917       -2     
  Branches        258      258              
============================================
- Hits           1359     1357       -2     
  Misses          443      443              
  Partials        117      117              
Impacted Files Coverage Δ
...n/java/seedu/intern/logic/commands/AddCommand.java 100.00% <ø> (ø)
.../java/seedu/intern/logic/commands/EditCommand.java 82.03% <ø> (-0.28%) :arrow_down:
...ava/seedu/intern/logic/commands/FilterCommand.java 75.00% <ø> (ø)
...a/seedu/intern/logic/parser/EditCommandParser.java 71.69% <ø> (ø)
...eedu/intern/model/applicant/ApplicationStatus.java 85.29% <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 46845a4...55bb50b. Read the comment docs.