-
Notifications
You must be signed in to change notification settings - Fork 86
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
fix: add error handling for LoadConfigOnce() #520
fix: add error handling for LoadConfigOnce() #520
Conversation
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## main #520 +/- ##
==========================================
+ Coverage 35.15% 36.50% +1.34%
==========================================
Files 29 29
Lines 1502 1504 +2
==========================================
+ Hits 528 549 +21
+ Misses 955 936 -19
Partials 19 19
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
LGTM
Signed-off-by: Junjie Gao <[email protected]>
Signed-off-by: Junjie Gao <[email protected]>
7f7718b
to
14e8173
Compare
Signed-off-by: Junjie Gao <[email protected]>
61b5fa8
to
2fa2a90
Compare
Signed-off-by: Junjie Gao <[email protected]>
77bc3af
to
860c382
Compare
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.
LGTM
Added error handling.
Added unit test.
#516
Resolves #525
Signed-off-by: Junjie Gao [email protected]