codecov-commenter
commented
1 year ago Codecov Report
Patch coverage: 77.77% and project coverage change: -0.23 :warning:
Comparison is base (
0751b55) 85.26% compared to head (4bb6c15) 85.04%.
Additional details and impacted files
```diff @@ Coverage Diff @@ ## main #69 +/- ## ========================================== - Coverage 85.26% 85.04% -0.23% ========================================== Files 10 10 Lines 699 702 +3 ========================================== + Hits 596 597 +1 - Misses 103 105 +2 ``` | [Impacted Files](https://codecov.io/gh/JKRhb/dtls2/pull/69?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jan+Romann) | Coverage Ξ | | |---|---|---| | [lib/src/dtls\_client.dart](https://codecov.io/gh/JKRhb/dtls2/pull/69?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jan+Romann#diff-bGliL3NyYy9kdGxzX2NsaWVudC5kYXJ0) | `81.51% <75.00%> (-0.70%)` | :arrow_down: | | [lib/src/dtls\_server.dart](https://codecov.io/gh/JKRhb/dtls2/pull/69?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jan+Romann#diff-bGliL3NyYy9kdGxzX3NlcnZlci5kYXJ0) | `87.78% <80.00%> (ΓΈ)` | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jan+Romann). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jan+Romann):umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
As requested by @Ifilehk (π), this PR exposes the state of
DtlsConnections, making it possible to check for library users to perform a slightly more fine-grained inspection of both client and server-side connections.As mentioned in #66, there is still room for improvement when it comes to state management, but for now, I think this PR should be fine to merge.