Skip to content

Commit

Permalink
Merge pull request #6055 from KKoukiou/expand-refactoring-pylint
Browse files Browse the repository at this point in the history
Fix useless-return pylint rule
  • Loading branch information
KKoukiou authored Dec 18, 2024
2 parents bbf33a0 + 42bf526 commit e9009aa
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 4 deletions.
3 changes: 0 additions & 3 deletions pyanaconda/gnome_remote_desktop.py
Original file line number Diff line number Diff line change
Expand Up @@ -147,9 +147,6 @@ def _find_network_address(self):
time.sleep(1)
tries -= 1

if not self.ip:
return

def _get_hostname(self):
"""Start thread to obtain hostname from DNS server asynchronously.
Expand Down
1 change: 1 addition & 0 deletions pyanaconda/ui/tui/spokes/shell_spoke.py
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,7 @@ def refresh(self, args=None):
NormalTUISpoke.refresh(self, args)
self.window.add_with_separator(TextWidget(_("Exit the shell to continue")))

# pylint: disable-next=useless-return
def prompt(self, args=None):
# run shell instead of printing prompt and close window on shell exit
execConsole()
Expand Down
1 change: 0 additions & 1 deletion tests/pylint/pylintrc
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,6 @@ disable=protected-access,
too-many-public-methods,
too-many-return-statements,
too-many-statements,
useless-return,
# End R (refactoring) whitelist

# Enable the message, report, category or checker with the given id(s). You can
Expand Down

0 comments on commit e9009aa

Please sign in to comment.