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: Reconfigure package imports #97

Merged

Conversation

robberwick
Copy link
Collaborator

This pull request includes changes to the src/blinkstick/__init__.py file to improve module imports and organization. The most important changes include adding imports for key classes and functions, and removing an unnecessary import.

Improvements to module imports:

  • Added imports for BlinkStick, BlinkStickPro, BlinkStickProMatrix, find_all, find_by_serial, and find_first from the blinkstick module.
  • Added imports for Color and ColorFormat from the colors module.
  • Added import for BlinkStickVariant from the constants module.
  • Added import for BlinkStickException from the exceptions module.

Codebase simplification:

  • Removed the import for main from the main module.Improve the discoverability of the key parts of the blinkstick package by exposing them at the top level.

@robberwick robberwick force-pushed the reconfigure-package-exports branch from af5a280 to b5da5a7 Compare November 10, 2024 08:30
Improve the discoverability of the key parts of the blinkstick package by exposing them at the top level.
Fix up the main script imports to use the new project structure, and update the entrypoint in pyproject.toml
@robberwick robberwick force-pushed the reconfigure-package-exports branch from b5da5a7 to 3220403 Compare November 10, 2024 08:50
@robberwick robberwick merged commit 3c6b842 into arvydas:release/2.0-dev Nov 10, 2024
15 checks passed
@robberwick robberwick deleted the reconfigure-package-exports branch November 10, 2024 08:53
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