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

tcp: Allow unknown0 to be 0 in addition to UINT16_MAX #198

Merged
merged 1 commit into from
May 31, 2022

Conversation

MarijnS95
Copy link
Contributor

As it turns out this field has a different value on a 4000TL(2?): while it doesn't bring us any closer to the meaning of this field, allow it to not spam the logs full with it.

As it turns out this field has a different value on a 4000TL(2?): while
it doesn't bring us any closer to the meaning of this field, allow it to
not spam the logs full with it.
@klaasnicolaas klaasnicolaas added the bugfix Fixing a bug. label May 31, 2022
@codecov
Copy link

codecov bot commented May 31, 2022

Codecov Report

Merging #198 (cd994bf) into main (928802e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #198   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            6         6           
  Lines          325       325           
  Branches        65        65           
=========================================
  Hits           325       325           
Impacted Files Coverage Δ
omnikinverter/tcp.py 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 928802e...cd994bf. Read the comment docs.

@klaasnicolaas klaasnicolaas merged commit 6a36cf3 into klaasnicolaas:main May 31, 2022
@MarijnS95 MarijnS95 deleted the tcp-unknown0-0 branch May 31, 2022 14:47
@github-actions github-actions bot locked and limited conversation to collaborators Jun 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bugfix Fixing a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants