-
Notifications
You must be signed in to change notification settings - Fork 82
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
imp(core): remove State::Uninitialized
check while parsing ConnectionEnd
#1126
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1126 +/- ##
==========================================
+ Coverage 66.46% 66.50% +0.04%
==========================================
Files 209 209
Lines 20706 20692 -14
==========================================
Hits 13762 13762
+ Misses 6944 6930 -14 ☔ View full report in Codecov by Sentry. |
.changelog/unreleased/improvements/1123-remove-check-for-unitialized-connection-end-state.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Just a small changelog nit
…alized-connection-end-state.md Co-authored-by: Farhad Shabani <[email protected]> Signed-off-by: Rano | Ranadeep <[email protected]>
…ionEnd` (#1126) * rm check for uninitialized connection end state * add changelog * Update .changelog/unreleased/improvements/1123-remove-check-for-unitialized-connection-end-state.md Co-authored-by: Farhad Shabani <[email protected]> Signed-off-by: Rano | Ranadeep <[email protected]> * md fmt --------- Signed-off-by: Rano | Ranadeep <[email protected]> Co-authored-by: Farhad Shabani <[email protected]>
Closes: #1123
Description
PR author checklist:
unclog
.docs/
).Reviewer checklist:
Files changed
in the GitHub PR explorer.