Skip to content

chore: deprecate client utils#22

Merged
aaron-steinfeld merged 3 commits intomainfrom
deprecate-client-utils
Oct 22, 2021
Merged

chore: deprecate client utils#22
aaron-steinfeld merged 3 commits intomainfrom
deprecate-client-utils

Conversation

@aaron-steinfeld
Copy link
Copy Markdown
Contributor

Description

Deprecating something that has long since been replaced (we just forgot to deprecate it, causing confusion).

tim-mwangi
tim-mwangi previously approved these changes Oct 22, 2021
@github-actions

This comment has been minimized.

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 22, 2021

Codecov Report

Merging #22 (f3eeef6) into main (cb552a0) will increase coverage by 0.09%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main      #22      +/-   ##
============================================
+ Coverage     72.44%   72.54%   +0.09%     
  Complexity      101      101              
============================================
  Files            17       17              
  Lines           294      295       +1     
  Branches         21       21              
============================================
+ Hits            213      214       +1     
  Misses           64       64              
  Partials         17       17              
Flag Coverage Δ
unit 72.54% <100.00%> (+0.09%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...grpcutils/client/GrpcClientRequestContextUtil.java 91.66% <100.00%> (+0.75%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cb552a0...f3eeef6. Read the comment docs.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@aaron-steinfeld aaron-steinfeld merged commit 66295c1 into main Oct 22, 2021
@aaron-steinfeld aaron-steinfeld deleted the deprecate-client-utils branch October 22, 2021 14:32
@github-actions
Copy link
Copy Markdown

Unit Test Results

12 files  ±0  12 suites  ±0   17s ⏱️ +2s
55 tests ±0  55 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 66295c1. ± Comparison against base commit cb552a0.

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.

3 participants