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

Bump okhttp dependency, exclude vulnerable transitives #1394

Merged
merged 3 commits into from
Aug 16, 2023

Conversation

dotasek
Copy link
Collaborator

@dotasek dotasek commented Aug 15, 2023

No description provided.

@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.01% ⚠️

Comparison is base (5925306) 11.59% compared to head (7c5314c) 11.58%.
Report is 1 commits behind head on master.

❗ Current head 7c5314c differs from pull request most recent head 1711abc. Consider uploading reports for the commit 1711abc to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1394      +/-   ##
============================================
- Coverage     11.59%   11.58%   -0.01%     
- Complexity    28755    28757       +2     
============================================
  Files          2125     2125              
  Lines        645055   645055              
  Branches     189258   189258              
============================================
- Hits          74763    74760       -3     
- Misses       542686   542690       +4     
+ Partials      27606    27605       -1     

see 3 files with indirect coverage changes

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

@grahamegrieve
Copy link
Collaborator

This failed because of an issue in the terminology server

@grahamegrieve grahamegrieve merged commit 96497d2 into master Aug 16, 2023
30 checks passed
@grahamegrieve grahamegrieve deleted the do-20230815-bump-okhttp-dependencies branch August 16, 2023 09:26
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