dwyl / dwylbot

:robot: Automating our GitHub Workflow to improve team communication/collaboration and reduce tedious repetition!
28 stars 7 forks source link

Add Credo to the pre-commit #44

Closed SimonLab closed 7 years ago

SimonLab commented 7 years ago

see #34

I've also refactor a few part of the code to match Credo style guide.

codecov-io commented 7 years ago

Codecov Report

Merging #44 into master will not change coverage. The diff coverage is 69.23%.

Impacted file tree graph

@@         Coverage Diff          @@
##           master   #44   +/-   ##
====================================
  Coverage      82%   82%           
====================================
  Files           8     8           
  Lines          50    50           
====================================
  Hits           41    41           
  Misses          9     9
Impacted Files Coverage Δ
web/controllers/repo_controller.ex 0% <0%> (ø) :arrow_up:
web/controllers/webhooks_controller.ex 100% <100%> (ø) :arrow_up:
web/models/user_from_auth.ex 100% <100%> (ø) :arrow_up:
web/controllers/auth.ex 85.71% <100%> (ø) :arrow_up:

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 2a4b6bd...6674ba8. Read the comment docs.

samhstn commented 7 years ago

Looks good 👍