flake8
-
reviewboard/oauth/models.py (Diff revision 1) Show all issues -
Review Request #8987 — Created June 2, 2017 and submitted
Information | |
---|---|
brennie | |
Review Board | |
release-3.0.x | |
|
|
9002 | |
Reviewers | |
reviewboard | |
The OAuth2 application form now makes use of some new widgets to make
the user experience better. The logic for the form validation has also
been revamped to take into account when validation of the redirect URIs
field fails (since they will no longer be present in the form's cleaned
data).
Manually tested the form.
Ran unit tests.
Description | From | Last Updated |
---|---|---|
Should be "Client ID" |
|
|
Not very helpful. It's just a repeat of the label. Can you update this to say exactly what this is? |
|
|
W293 blank line contains whitespace |
![]() |
|
W291 trailing whitespace |
![]() |
|
Blank line before conditional. |
|
|
There seems to be a mismatched <? |
|
|
g before m |
|
|
E261 at least two spaces before inline comment |
![]() |
|
Swap these. |
|
|
Blank line between these. |
|
|
This can be one statement now. |
|
|
I imagine we're doing this because the parent model does clean and we don't want that one to run? It's … |
|
|
clean -> clean() |
|
|
Swap these. |
|
Add unit tests.
Testing Done: |
|
|||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+150 -9) |
reviewboard/accounts/forms/pages.py (Diff revision 5) |
---|
E261 at least two spaces before inline comment
reviewboard/oauth/models.py (Diff revision 6) |
---|
I imagine we're doing this because the parent model does
clean
and we don't want that one to run? It's not entirely clear from the description (context is missing).