Closed pc2752 closed 4 years ago
Hello @pc2752! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
tests/test_mediaobject.py
:Line 20:9: E128 continuation line under-indented for visual indent Line 20:80: E501 line too long (99 > 79 characters) Line 68:9: E128 continuation line under-indented for visual indent Line 68:80: E501 line too long (99 > 79 characters) Line 79:9: E128 continuation line under-indented for visual indent Line 79:80: E501 line too long (155 > 79 characters) Line 79:155: E502 the backslash is redundant between brackets Line 80:9: E128 continuation line under-indented for visual indent Line 84:9: E128 continuation line under-indented for visual indent Line 84:80: E501 line too long (170 > 79 characters) Line 84:170: E502 the backslash is redundant between brackets Line 85:9: E128 continuation line under-indented for visual indent
trompace/mutations/mediaobject.py
:Line 25:76: E231 missing whitespace after ':' Line 25:80: E501 line too long (100 > 79 characters) Line 25:101: W291 trailing whitespace Line 68:80: E501 line too long (123 > 79 characters) Line 68:111: E231 missing whitespace after ':' Line 68:124: W291 trailing whitespace
trompace/queries/mediaobject.py
:Line 9:17: E128 continuation line under-indented for visual indent Line 9:80: E501 line too long (110 > 79 characters) Line 9:99: E231 missing whitespace after ':' Line 9:111: W291 trailing whitespace Line 21:80: E501 line too long (106 > 79 characters)
inLanguage was present in the docstring, but not in the arguments of the function, guess it went missing at some point. Added for both create and update mutations and the queries.