[freeipa PR#2136][opened] [Backport][ipa-4-6] Fix pylint 2.0 return-related violations

[tiran]

URL: https://github.com/freeipa/freeipa/pull/2136
Author: tiran
 Title: #2136: [Backport][ipa-4-6] Fix pylint 2.0 return-related violations
Action: opened

PR body:
"""
Manual backport of PR #2135 to 4.6 branch.

Aiming to support pylint 2.0 some functions and methods must have their
return statements updated in order to fix two new violations:

- `useless-return` (R1711):
  Useless return at end of function or method Emitted when a single
  "return" or "return None" statement is found at the end of function
  or method definition. This statement can safely be removed because
  Python will implicitly return None

- `inconsistent-return-statements` (R1710):
  Either all return statements in a function should return an
  expression, or none of them should. According to PEP8, if any return
  statement returns an expression, any return statements where no value
  is returned should explicitly state this as return None, and an
  explicit return statement should be present at the end of the
  function (if reachable)

Issue: https://pagure.io/freeipa/issue/7614

Signed-off-by: Armando Neto 
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/2136/head:pr2136
git checkout pr2136

[freeipa PR#2136][closed] [Backport][ipa-4-6] Fix pylint 2.0 return-related violations

[tiran]

URL: https://github.com/freeipa/freeipa/pull/2136
Author: tiran
 Title: #2136: [Backport][ipa-4-6] Fix pylint 2.0 return-related violations
Action: closed

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/2136/head:pr2136
git checkout pr2136