Skip to content
This repository has been archived by the owner on Aug 30, 2022. It is now read-only.

Document that the Thrift generated files are patched #50

Merged
merged 1 commit into from
Feb 1, 2018

Conversation

ringerc
Copy link
Contributor

@ringerc ringerc commented Jan 31, 2018

For #45, document that the code-gen output from the IDL is currently patched from the original generated code, and why.

@codecov
Copy link

codecov bot commented Jan 31, 2018

Codecov Report

Merging #50 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #50   +/-   ##
=======================================
  Coverage   88.46%   88.46%           
=======================================
  Files          93       93           
  Lines        2246     2246           
=======================================
  Hits         1987     1987           
  Misses        259      259

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 2e22904...b2dc8ad. Read the comment docs.

@yurishkuro
Copy link
Member

@ringerc could you please amend the commit to make it signed? git commit --amend -s

@ringerc ringerc force-pushed the document-patched-idl branch from 03219f0 to b2dc8ad Compare January 31, 2018 06:12
@ringerc
Copy link
Contributor Author

ringerc commented Jan 31, 2018

@yurishkuro Done

@isaachier
Copy link
Contributor

Awesome work! Thank you for the docs.

@isaachier isaachier merged commit c36adcf into jaegertracing:master Feb 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants