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

Refactor: Move settings and GUI related code to a separate module #27

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

RedAtman
Copy link
Owner

This commit moves the settings and GUI-related functionality into a new module named gui.py. This improves the organization and separation of concerns within the project, making it easier to maintain and understand.

The following changes were made:

  • The settings.py module was removed, and its contents were moved to gui.py.
  • The utils/sublime.py module was renamed to gui.py.
  • The api.py, models.py, and operations.py modules were updated to import the necessary functions from gui.py.

This refactoring makes the codebase more modular and easier to understand. It also ensures that the GUI-related logic is separated from the core application logic.

This commit moves the settings and GUI-related functionality into a new module named `gui.py`. This improves the organization and separation of concerns within the project, making it easier to maintain and understand.

The following changes were made:

- The `settings.py` module was removed, and its contents were moved to `gui.py`.
- The `utils/sublime.py` module was renamed to `gui.py`.
- The `api.py`, `models.py`, and `operations.py` modules were updated to import the necessary functions from `gui.py`.

This refactoring makes the codebase more modular and easier to understand. It also ensures that the GUI-related logic is separated from the core application logic.
@RedAtman RedAtman merged commit 1ea994f into master Jul 18, 2024
1 check passed
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