Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

break up imports loops except for calls to GUI functions from logitech_receiver code #2265

Merged
merged 5 commits into from
Feb 18, 2024

Conversation

pfps
Copy link
Collaborator

@pfps pfps commented Feb 11, 2024

Adresses #2262

@pfps pfps changed the title break up an imports loop and do some other refactoring break up imports loops inside logitech_receiver Feb 11, 2024
@pfps pfps changed the title break up imports loops inside logitech_receiver break up imports loops except for calls to GUI functions from logitech_receiver code Feb 11, 2024
@pfps pfps closed this Feb 15, 2024
@pfps pfps deleted the imports branch February 15, 2024 15:38
@pfps pfps restored the imports branch February 15, 2024 15:38
@pfps pfps reopened this Feb 16, 2024
pfps added 5 commits February 18, 2024 06:18
device: move some imports to top of modules

device: break up imports loop with device descriptors

device: break up imports loop by moving a function from notifications.py to setting_templates.py

device: break import loop between device.py and diversion.py by using device to access method
ui: move imports in about.py to top of file

ui: move imports to top of notify.py

ui: move imports to top of window.py

ui: reorder imports at beginning of __init__.py

ui: move imports to top of tray.py

ui: move common code out of __init__.py to common.py

ui: move imports to top of __init___.py
solaar: move imports to top of files

solaar: move more imports to top of files

solaar: guard Gtk, etc imports with correct version
device: move imports of ui modules to beginning of files

logitech_receiver: remove imports from __init__.py
@pfps pfps merged commit 12de240 into pwr-Solaar:master Feb 18, 2024
5 checks passed
@pfps pfps deleted the imports branch February 18, 2024 11:21
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