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

Many device values can return None #64

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stefanor
Copy link
Contributor

@stefanor stefanor commented Oct 23, 2024

Maybe some should return infinity, but at least let the type hint reflect reality.

@stefanor stefanor force-pushed the get_remaining_mins-optional branch 2 times, most recently from 13b2442 to 2eabcd0 Compare October 23, 2024 23:41
@stefanor stefanor changed the title get_remaining_mins() can return None Many battery monitor values can return None Oct 23, 2024
Declare them as such
@stefanor stefanor force-pushed the get_remaining_mins-optional branch from 2eabcd0 to e9a37ab Compare October 24, 2024 00:17
@stefanor stefanor changed the title Many battery monitor values can return None Many device values can return None Oct 24, 2024
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