-
Notifications
You must be signed in to change notification settings - Fork 4
Refactor and Improve Type Hints, Clean Up Code, and Remove requests Attribute from StatusData #12
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Introduced type variables for better type hinting (zsh: command not found: T, zsh: command not found: AsyncCallable). - Updated method signatures to use specific types instead of forward declarations (zsh: command not found: Client). - Enhanced decorator typings to accept async functions with appropriate return types. - Removed deprecated zsh: command not found: data method and applied decorators to zsh: command not found: logs method. - Added return type annotations for several async methods (e.g., zsh: command not found: set_custom_domain, zsh: command not found: all_backups).
- Renamed zsh: command not found: extra to zsh: command not found: extra_value in notifications for consistency. - Removed the commented-out zsh: command not found: app_backups method and the deprecated zsh: command not found: app_data method. - Refactored app data handling in zsh: command not found: get_app_data to simplify language handling. - Updated method signatures to reflect accurate return types and removed unnecessary comments. - Cleaned up zsh: command not found: last_deploys method to ensure better clarity and maintainability.
- Replaced decorator with a custom metaclass zsh: command not found: DataClasMeta for better flexibility and control. - Renamed zsh: command not found: language field to zsh: command not found: lang in zsh: command not found: AppData and zsh: command not found: StatusData for consistency. - Removed unused zsh: command not found: requests field in zsh: command not found: StatusData. - Simplified class inheritance structure for better readability and maintainability.
- Changed zsh: command not found: extra_value back to zsh: command not found: extra in the zsh: command not found: RequestListenerManager class to maintain consistency with the rest of the codebase.
- Added explicit return types for zsh: command not found: run_async_script and zsh: command not found: delete_test_apps. - Updated type hints in zsh: command not found: run_async_script for better typing consistency. - Fixed emoji code formatting in zsh: command not found: delete_test_apps print statements.
- Add precise type annotations to fixtures in zsh: command not found: conftest.py - Use zsh: command not found: AsyncGenerator in zsh: command not found: app fixture for clarity - Remove unused test cases: zsh: command not found: test_app_data and zsh: command not found: test_request_app_data - Standardize zsh: command not found: @pytest.mark.skipif formatting - Enhance test readability and streamline coverage
…le to define project metadata for safety checks
…ling for improved development, linting, and security - Updated Python requirement to 3.13 - Updated dependencies for better compatibility - Added Bandit configuration for security checks - Included 'ruff check .' in the lint command - Fixed package inclusion quotes for consistency
…onsistency and readability
…or consistency across the file
- Removed unused 'requests' variable from StatusData - Reformatted type hints in deviceTypes and operatingSystems for better readability
…syntax >>> ⏰ - Replaced single quotes with double quotes for consistency - Removed unused import statements - Updated dictionary keys and values to use double quotes
…i SDK from README to streamline documentation
Contributor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 72 out of 73 changed files in this pull request and generated 17 comments.
Files not reviewed (1)
- .safety-project.ini: Language not supported
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changelog
Chores
py.typedfile to indicate the package supports type hints. ([38d4e86](robert-nogueira@38d4e86)).safety-project.iniconfiguration file to define project metadata for safety checks. ([ee37f1d](robert-nogueira@ee37f1d))Tests
Styles
__kwargsto_kwargsfor consistency and readability. ([576e8c1](robert-nogueira@576e8c1))Refactors
RequestListenerManager. ([adcdff0](robert-nogueira@adcdff0))Enhancements
Documentation
BREAKING CHANGES
requestsattribute fromStatusData. ([926026be](robert-nogueira@926026be))