jazzband / django-rest-knox

Authentication Module for django rest auth
MIT License
1.1k stars 206 forks source link

test: Set up pre-commit hooks and fix quality nits #308

Closed giovannicimolin closed 10 months ago

giovannicimolin commented 10 months ago

This PR implements:

giovannicimolin commented 10 months ago

Relates to #304.

codecov[bot] commented 10 months ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@efcfdcb). Click here to learn what that means. The diff coverage is n/a.

:exclamation: Current head b55402e differs from pull request most recent head 5b936a7. Consider uploading reports for the commit 5b936a7 to get more accurate results

@@            Coverage Diff             @@
##             develop     #308   +/-   ##
==========================================
  Coverage           ?   90.30%           
==========================================
  Files              ?        9           
  Lines              ?      227           
  Branches           ?       35           
==========================================
  Hits               ?      205           
  Misses             ?       19           
  Partials           ?        3           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

giovannicimolin commented 10 months ago

It worked! Still needs some improvements. I'll do them on a follow-up ticket. :)