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

Jump to bottom

fix(log): simplify `DeviceDisconnectedError` error message #148

Merged

palazzem merged 1 commit into main from palazzem/disconnect-message

Mar 1, 2024

Owner

palazzem commented Mar 1, 2024

Related Issues

Follow-up for feat(api): handle main unit disconnections #147

Proposed Changes:

Simplify exception message.

Testing:

n/a

Extra Notes (optional):

n/a

Checklist

Related issues and proposed changes are filled
Tests are defining the correct and expected behavior
Code is well-documented via docstrings


          fix(log): simplify DeviceDisconnectedError log message

d73e852

palazzem added topic:api type:fix labels

palazzem added this to the 0.12.0 milestone

palazzem self-assigned this

palazzem merged commit 3b60eb6 into main

3 checks passed

palazzem deleted the palazzem/disconnect-message branch

March 1, 2024 13:28

coveralls commented Mar 1, 2024

Pull Request Test Coverage Report for Build 8112108230

Details

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 8110498604:	0.0%
Covered Lines:	393
Relevant Lines:	393

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic:api type:fix