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

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

mc/zw/an-6pm 3 tutor notes database implementation #242

Closed m-cabello closed 3 years ago

m-cabello commented 3 years ago

Implemented a tutor notes database and columns exist for course, author/uploader, and the notes.

github-actions[bot] commented 3 years ago

Nice PR! Your PR can be merged once you get an approving review from a staff member and a member of your team.

Keep in mind, code reviews only count if you use the "Approve" or "Request changes" option in GitHub. You can find this under the "Files changed" tab under the "Review changes" button.

PR review process

Generated by :no_entry_sign: dangerJS against 859b1c7e25d00a6a03d184c03559185ad006d3a6

codecov[bot] commented 3 years ago

Codecov Report

Merging #242 (859b1c7) into main (221ec4a) will increase coverage by 0.00%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #242   +/-   ##
=========================================
  Coverage     99.82%   99.82%           
- Complexity      342      357   +15     
=========================================
  Files            83       84    +1     
  Lines          1669     1701   +32     
  Branches         77       78    +1     
=========================================
+ Hits           1666     1698   +32     
  Misses            3        3           
Impacted Files Coverage Δ
...n/java/edu/ucsb/ucsbcslas/entities/TutorNotes.java 100.00% <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 221ec4a...859b1c7. Read the comment docs.

pconrad commented 3 years ago

Estimate this at 10 points

pconrad commented 3 years ago

LGTM, good works

This is a comment, not a code review. Please do a code review.