URL:
https://github.com/SSSD/sssd/pull/527
Author: mzidek-rh
Title: #527: PEP8 Fixes and enable PEP8 check in CI
Action: opened
PR body:
"""
Hi,
because @fidencio is on PTO I allowed myself to rebase and update his branch:
https://github.com/fidencio/sssd/tree/wip/ci_enable_pep8_check
and present it here as new PR.
Here is link to the ticket:
https://pagure.io/SSSD/sssd/issue/3605
What I did:
- reviewed all the patches that were not reviewed yet (some were already reviewed)
- rebased them on top of current master (there were only minor conflicts)
- if there are some missing patches that is because the issue that they fixed was no
longer present in the current master, so I deleted them
- added "Resolves" and "Reviewed" tags (so the person that pushes it
does not need to add them)
- pushed the patchset to CI (passed)
- added pep8 issue and tried the CI (issue was detected on all platforms)
The patchest is now fully ACKed.
Note: because of the way I rebased the patches, some are not in the original order
"""
To pull the PR as Git branch:
git remote add ghsssd
https://github.com/SSSD/sssd
git fetch ghsssd pull/527/head:pr527
git checkout pr527