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.

Sign up for GitHub

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

Type annotations and mypy #821

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Make mypy happier

49f6546
Select commit
Loading
Failed to load commit list.
Draft

Type annotations and mypy #821

Make mypy happier
49f6546
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jul 22, 2024 in 1s

53.04% of diff hit (target 21.52%)

View this Pull Request on Codecov

53.04% of diff hit (target 21.52%)

Annotations

Check warning on line 24 in jellyfin_kodi/client.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/client.py#L24 

Added line #L24 was not covered by tests

Check warning on line 28 in jellyfin_kodi/client.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/client.py#L28 

Added line #L28 was not covered by tests

Check warning on line 132 in jellyfin_kodi/connect.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/connect.py#L132 

Added line #L132 was not covered by tests

Check warning on line 148 in jellyfin_kodi/connect.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/connect.py#L147-L148 

Added lines #L147 - L148 were not covered by tests

Check warning on line 152 in jellyfin_kodi/connect.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/connect.py#L152 

Added line #L152 was not covered by tests

Check warning on line 157 in jellyfin_kodi/connect.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/connect.py#L157 

Added line #L157 was not covered by tests

Check warning on line 55 in jellyfin_kodi/dialogs/context.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/dialogs/context.py#L55 

Added line #L55 was not covered by tests

Check warning on line 77 in jellyfin_kodi/dialogs/context.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/dialogs/context.py#L77 

Added line #L77 was not covered by tests

Check warning on line 348 in jellyfin_kodi/entrypoint/default.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/entrypoint/default.py#L348 

Added line #L348 was not covered by tests

Check warning on line 888 in jellyfin_kodi/entrypoint/default.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/entrypoint/default.py#L888 

Added line #L888 was not covered by tests

Check warning on line 315 in jellyfin_kodi/entrypoint/service.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/entrypoint/service.py#L314-L315 

Added lines #L314 - L315 were not covered by tests

Check warning on line 47 in jellyfin_kodi/helper/api.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/helper/api.py#L47 

Added line #L47 was not covered by tests

Check warning on line 157 in jellyfin_kodi/helper/utils.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/helper/utils.py#L157 

Added line #L157 was not covered by tests

Check warning on line 170 in jellyfin_kodi/helper/utils.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/helper/utils.py#L170 

Added line #L170 was not covered by tests

Check warning on line 181 in jellyfin_kodi/helper/utils.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/helper/utils.py#L181 

Added line #L181 was not covered by tests

Check warning on line 75 in jellyfin_kodi/helper/wrapper.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/helper/wrapper.py#L75 

Added line #L75 was not covered by tests

Check warning on line 75 in jellyfin_kodi/jellyfin/__init__.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/jellyfin/__init__.py#L75 

Added line #L75 was not covered by tests

Check warning on line 53 in jellyfin_kodi/library.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/library.py#L51-L53 

Added lines #L51 - L53 were not covered by tests

Check warning on line 57 in jellyfin_kodi/library.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/library.py#L57 

Added line #L57 was not covered by tests

Check warning on line 62 in jellyfin_kodi/library.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/library.py#L62 

Added line #L62 was not covered by tests

Check warning on line 229 in jellyfin_kodi/library.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/library.py#L229 

Added line #L229 was not covered by tests

Check warning on line 120 in jellyfin_kodi/monitor.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/monitor.py#L120 

Added line #L120 was not covered by tests

Check warning on line 124 in jellyfin_kodi/monitor.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/monitor.py#L124 

Added line #L124 was not covered by tests

Check warning on line 133 in jellyfin_kodi/monitor.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/monitor.py#L133 

Added line #L133 was not covered by tests

Check warning on line 153 in jellyfin_kodi/monitor.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

jellyfin_kodi/monitor.py#L152-L153 

Added lines #L152 - L153 were not covered by tests
View more details on Codecov