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

Add temporary ignore for watchdog.Observer type hint #72

Merged
merged 1 commit into from Mar 25, 2023

Conversation

Callum027
Copy link
Member

A bug in the type hint for the watchdog.Observer base class makes Mypy fail.

This error does not show on the pre-commit hooks because not all the packages are installed into that environment.

Add an error-specific ignore comment to the offending line, and add a TODO to document the tracking issue in the watchdog repository.

gorakhargosh/watchdog#982

@Callum027 Callum027 added bug Something isn't working pipeline CI pipeline-related issue or pull request labels Mar 25, 2023
@Callum027 Callum027 added this to the v0.4.0 milestone Mar 25, 2023
@Callum027 Callum027 self-assigned this Mar 25, 2023
@Callum027 Callum027 merged commit 06de329 into main Mar 25, 2023
1 check passed
@Callum027 Callum027 deleted the x-observer-typeignore branch March 25, 2023 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working pipeline CI pipeline-related issue or pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant