Skip to content

Sourcery Starbot ⭐ refactored cundesi/UserNormalTranslator#1

Open
SourceryAI wants to merge 1 commit intocundesi:masterfrom
SourceryAI:master
Open

Sourcery Starbot ⭐ refactored cundesi/UserNormalTranslator#1
SourceryAI wants to merge 1 commit intocundesi:masterfrom
SourceryAI:master

Conversation

@SourceryAI
Copy link

Thanks for starring sourcery-ai/sourcery ✨ 🌟 ✨

Here's your pull request refactoring your most popular Python repo.

If you want Sourcery to refactor all your Python repos and incoming pull requests install our bot.

Review changes via command line

To manually merge these changes, make sure you're on the master branch, then run:

git fetch https://github.com/sourcery-ai-bot/UserNormalTranslator master
git merge --ff-only FETCH_HEAD
git reset HEAD^

preferred_order = list(
b for b in QT_PREFERRED_BINDING.split(os.pathsep) if b
)
preferred_order = [b for b in QT_PREFERRED_BINDING.split(os.pathsep) if b]
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Function _loadUi._UiLoader._loadCustomWidgets.headerToModule._import_sub_module._setup._pyside2._pyside._pyqt5._pyqt4._install refactored with the following changes:

  • Replace list(), dict() or set() with comprehension (collection-builtin-to-comprehension)


homedir = os.path.dirname(__file__)
if not homedir in sys.path:
if homedir not in sys.path:
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lines 14-14 refactored with the following changes:

  • Simplify logical expression using De Morgan identities (de-morgan)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant