Skip to content

Fix NPE when resolving state#401

Merged
S7evinK merged 2 commits intomainfrom
s7evink/panic
Jul 7, 2023
Merged

Fix NPE when resolving state#401
S7evinK merged 2 commits intomainfrom
s7evink/panic

Conversation

@S7evinK
Copy link
Collaborator

@S7evinK S7evinK commented Jul 7, 2023

No description provided.

@codecov
Copy link

codecov bot commented Jul 7, 2023

Codecov Report

Patch coverage: 28.57% and project coverage change: -0.99 ⚠️

Comparison is base (35f734f) 58.47% compared to head (017c9f5) 57.48%.

❗ Current head 017c9f5 differs from pull request most recent head 6ff4cb1. Consider uploading reports for the commit 6ff4cb1 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #401      +/-   ##
==========================================
- Coverage   58.47%   57.48%   -0.99%     
==========================================
  Files          51       51              
  Lines        6993     6969      -24     
==========================================
- Hits         4089     4006      -83     
- Misses       2517     2566      +49     
- Partials      387      397      +10     
Impacted Files Coverage Δ
stateresolutionv2.go 88.61% <28.57%> (-1.22%) ⬇️

... and 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@S7evinK S7evinK merged commit 226d208 into main Jul 7, 2023
@S7evinK S7evinK deleted the s7evink/panic branch July 7, 2023 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants