Closed BryanFauble closed 5 months ago
Hello @BryanFauble! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Line 212:89: E501 line too long (120 > 88 characters) Line 213:89: E501 line too long (116 > 88 characters) Line 214:89: E501 line too long (138 > 88 characters)
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code
Problem: The order of credentials being tried was incorrect in the comment
Solution: Correct the comment to match the code from: https://github.com/Sage-Bionetworks/synapsePythonClient/blob/develop/synapseclient/core/credentials/credential_provider.py#L258