-
-
Notifications
You must be signed in to change notification settings - Fork 7k
Add import example for force_authenticate in testing #2562
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Yah. |
Damnit, really need to pin pep8. |
Bit meh to these silent upgrades. We'll have to resolve it before we can merge anything else. Either pin pep8 to a lower version that's not so crazy about exactly where binary operators should be positioned, or by re-positioning 'em. Eg:
Sorry - nothing related to your PR of course, but does block it. |
Wow that's weird, I added a single line in the docs? |
@tomchristie raise this as an issue / separate PR? |
Testing against a newer version of pep8 than the previous commit/PR.
@phalt Yeah. |
Tox is pinning pep8 to |
I'll just fix the pep8 errors :) |
https://github.com/tomchristie/django-rest-framework/blob/master/tox.ini#L26 How can it not be running |
Wondering if now that we have #2568 this can be cleaned up to only include the missing import |
@jpadilla I'll rebase and push |
@tomchristie @jpadilla tah dah |
Add import example for force_authenticate in testing
@phalt thanks! |
Doing a bit of testing with authentication and I noticed there was no mention of where to import the
force_authenticate
method anywhere, so I've added it to the first example whereforce_authenticate
is introduced.Fixes PEP8 W503 errors