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

Fix unhandled asyncio.CancelledError during shutdown #766

Merged
merged 1 commit into from
Nov 7, 2023
Merged

Conversation

bachya
Copy link
Owner

@bachya bachya commented Nov 7, 2023

Describe what the PR does:

SSIA

Does this fix a specific issue?

N/A

Checklist:

  • Confirm that one or more new tests are written for the new functionality.
  • Run tests and ensure everything passes (with 100% test coverage).
  • Update README.md with any new documentation.

@bachya bachya added the bug Bugs or issues which will cause a problem for users label Nov 7, 2023
@bachya bachya self-assigned this Nov 7, 2023
Copy link

codecov bot commented Nov 7, 2023

Codecov Report

Merging #766 (ccf3659) into dev (00dd5a4) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               dev      #766   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           31        31           
  Lines         1765      1758    -7     
=========================================
- Hits          1765      1758    -7     
Files Coverage Δ
ecowitt2mqtt/runtime.py 100.00% <100.00%> (ø)

@bachya bachya merged commit c81fd98 into dev Nov 7, 2023
12 checks passed
@bachya bachya deleted the fix-shutdown branch November 7, 2023 04:15
@github-actions github-actions bot locked and limited conversation to collaborators Nov 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Bugs or issues which will cause a problem for users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant