Skip to content
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(job): check response for nil in (*CurlJob).DumpResponse #117

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

reugn
Copy link
Owner

@reugn reugn commented Mar 8, 2024

No description provided.

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 33.33333% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 94.53%. Comparing base (45b7905) to head (a0f16e8).

Files Patch % Lines
job/curl_job.go 33.33% 1 Missing and 1 partial ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
+ Coverage   94.21%   94.53%   +0.32%     
==========================================
  Files          19       19              
  Lines         950      952       +2     
==========================================
+ Hits          895      900       +5     
+ Misses         37       33       -4     
- Partials       18       19       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@reugn reugn merged commit a90c2c9 into master Mar 8, 2024
5 checks passed
@reugn reugn deleted the response-nil-check branch March 8, 2024 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants