Closed acrellin closed 3 years ago
Hello @acrellin! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
app/models.py
:Line 896:80: E501 line too long (93 > 79 characters)
Woops, botched a rebase and overwrote the original diff here.. Will update.
@stefanv @kmshin1397 Updated, ready for review
@stefanv how does this look to you now?
Let's not merge this until we get the necessary updates in place in SP