opentelemetry-rust icon indicating copy to clipboard operation
opentelemetry-rust copied to clipboard

build: bump thrift to 0.16

Open TennyZhuang opened this issue 3 years ago • 3 comments

Signed-off-by: TennyZhuang [email protected]

Thrift forget to bump the version on crates.io for a long time, and recently they do it.

TennyZhuang avatar Jul 04 '22 06:07 TennyZhuang

CLA Not Signed

Codecov Report

Merging #832 (0ae7a8a) into main (20e46ee) will not change coverage. The diff coverage is n/a.

@@          Coverage Diff          @@
##            main    #832   +/-   ##
=====================================
  Coverage   68.7%   68.7%           
=====================================
  Files        115     115           
  Lines       9397    9397           
=====================================
  Hits        6464    6464           
  Misses      2933    2933           

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 20e46ee...0ae7a8a. Read the comment docs.

codecov[bot] avatar Jul 04 '22 06:07 codecov[bot]

I’d like to do it, but I always failed to sign with DocSign on every browser, including Safari and latest Chrome. It always warns me that required fields are not filled, but in fact I filled them correctly. Is this a known issue?

Anyway, the change is small, anyone can feel free to help me to resubmit the commit in another PR.

TennyZhuang avatar Jul 05 '22 18:07 TennyZhuang

This has been resolved in #848

jtescher avatar Sep 12 '22 00:09 jtescher