Closed osamingo closed 2 years ago
I would appreciate it if you would approve running GitHub Actions for this PR when you have time.
@Widcket ping
@sergiught I fixed it, PTAL.
Base: 89.05% // Head: 89.09% // Increases project coverage by +0.03%
:tada:
Coverage data is based on head (
2bc0d47
) compared to base (e9dbfc6
). Patch coverage: 100.00% of modified lines in pull request are covered.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Hey @osamingo 👋🏻 thanks a lot for the changes and apologies it took so while to get back to you! Would you mind adding a test case for this as well in the validatior_test
.go file? 🙏🏻
Hey @osamingo 👋🏻 I had some time and went ahead and added the test cases missing. Thanks for the contribution again!
Description
The threat of panicking should be curbed by receiving a second return value when executing the type-cast.
References
Testing
Checklist
master
.