Closed BryanFauble closed 8 months ago
Hello @BryanFauble! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
synapseclient/__main__.py
:Line 666:89: E501 line too long (90 > 88 characters) Line 806:89: E501 line too long (91 > 88 characters) Line 1048:89: E501 line too long (89 > 88 characters) Line 1223:89: E501 line too long (89 > 88 characters) Line 1235:89: E501 line too long (89 > 88 characters) Line 1673:89: E501 line too long (91 > 88 characters)
synapseclient/client.py
:Line 428:89: E501 line too long (93 > 88 characters) Line 433:89: E501 line too long (99 > 88 characters) Line 548:89: E501 line too long (90 > 88 characters) Line 556:89: E501 line too long (90 > 88 characters) Line 2040:89: E501 line too long (90 > 88 characters)
Line 40:89: E501 line too long (89 > 88 characters) Line 48:89: E501 line too long (90 > 88 characters) Line 169:89: E501 line too long (89 > 88 characters) Line 170:89: E501 line too long (90 > 88 characters) Line 178:89: E501 line too long (90 > 88 characters) Line 214:89: E501 line too long (116 > 88 characters) Line 215:89: E501 line too long (138 > 88 characters) Line 216:89: E501 line too long (120 > 88 characters) Line 217:89: E501 line too long (134 > 88 characters)
tests/integration/synapseclient/integration_test.py
:Line 13:1: F811 redefinition of unused 'Synapse' from line 12 Line 40:32: E711 comparison to None should be 'if cond is None:'
Line 150:89: E501 line too long (96 > 88 characters)
Kudos, no new issues were introduced!
0 New issues
0 Security Hotspots
97.8% Coverage on New Code
0.0% Duplication on New Code
Problem:
Solution:
Testing:
synapse login
andsynapse config
)