[Freeipa-devel] [freeipa PR#145][comment] [WIP] Refactoring: LDAP Connection Management

mbasti-rh freeipa-github-notification at redhat.com
Mon Oct 10 15:51:45 UTC 2016


  URL: https://github.com/freeipa/freeipa/pull/145
Title: #145: [WIP] Refactoring: LDAP Connection Management

mbasti-rh commented:
"""
Please fix PEP8
```
./ipapython/ipaldap.py:766:49: E231 missing whitespace after ','
./ipapython/ipaldap.py:822:9: E265 block comment should start with '# '
./ipaserver/install/bindinstance.py:231:1: E302 expected 2 blank lines, found 1
./ipaserver/install/bindinstance.py:231:80: E501 line too long (82 > 79 characters)
./ipaserver/install/krbinstance.py:170:80: E501 line too long (89 > 79 characters)
./ipaserver/install/ldapupdate.py:59:80: E501 line too long (85 > 79 characters)
./ipaserver/install/replication.py:218:80: E501 line too long (82 > 79 characters)
```


"""

See the full comment at https://github.com/freeipa/freeipa/pull/145#issuecomment-252662459


More information about the Freeipa-devel mailing list