search

ucsb-cs156-s21 / proj-ucsb-cs-las

https://proj-ucsb-cs-las.herokuapp.com/
MIT License
2 stars 3 forks source link

AL - Remove commented code from #68 #153

Closed andrewhlu closed 3 years ago

andrewhlu commented 3 years ago

I noticed that there's a lot of commented code in PR #68 that didn't get removed despite review comments stating to do so. This small PR removes that commented code.

codecov[bot] commented 3 years ago

Codecov Report

Merging #153 (cbaaa77) into main (ea35df4) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                main      #153    +/-   ##
============================================
  Coverage     100.00%   100.00%            
============================================
  Files             77        52    -25     
  Lines           1468       641   -827     
  Branches          69         6    -63     
============================================
- Hits            1468       641   -827
Impacted Files Coverage Δ Complexity Δ
javascript/src/main/pages/Tutor/Tutor.js 100.00% <ø> (ø) 0.00 <0.00> (ø)
.../edu/ucsb/ucsbcslas/controllers/AppController.java
...du/ucsb/ucsbcslas/advice/AuthControllerAdvice.java
...c/main/java/edu/ucsb/ucsbcslas/entities/Tutor.java
...c/main/java/edu/ucsb/ucsbcslas/entities/Admin.java
.../main/java/edu/ucsb/ucsbcslas/DemoApplication.java
.../main/java/edu/ucsb/ucsbcslas/documents/Login.java
...rc/main/java/edu/ucsb/ucsbcslas/models/Course.java
...csbcslas/models/TutorAssignmentOfficeHourView.java
...u/ucsb/ucsbcslas/controllers/CourseController.java
... and 16 more

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 ea35df4...cbaaa77. Read the comment docs.