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

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

ci: add Danger checks for common PR issues #227

Closed btk5h closed 3 years ago

btk5h commented 3 years ago

This PR adds a GitHub Actions workflow that checks for common mistakes in PRs and provides automated feedback about what issues were found and how the contributor can fix their PR.

To see the workflow in action, see #226

codecov[bot] commented 3 years ago

Codecov Report

Merging #227 (87afcbb) into main (7817a49) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #227   +/-   ##
=========================================
  Coverage     99.82%   99.82%           
  Complexity      342      342           
=========================================
  Files            83       83           
  Lines          1669     1669           
  Branches         77       77           
=========================================
  Hits           1666     1666           
  Misses            3        3           

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 7817a49...87afcbb. Read the comment docs.

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 87afcbb0a07bbca07fd0e1eac0052c2f446e3ec1