search

jazzband / django-rest-knox

Authentication Module for django rest auth
MIT License
1.15k stars 211 forks source link

Avoid importing `django.test` in main code #335

Closed angusholder closed 5 months ago

angusholder commented 5 months ago

setting_changed is actually from django.core.signals, and is re-exported from django.test.signals.

django.test takes 20-40ms to import, so not too much, but it's an easy win to avoid this import.

giovannicimolin commented 5 months ago

@angusholder Thanks for your contribution!

That's a nice catch! I can accept that right away.