Skip to content
This repository has been archived by the owner on May 23, 2024. It is now read-only.

chore(docs): fixed a couple of typos #475

Merged
merged 2 commits into from
Dec 16, 2019

Conversation

sircelsius
Copy link
Contributor

@sircelsius sircelsius commented Dec 13, 2019

Signed-off-by: Marc Bramaud m.duboucheron@gmail.com

Which problem is this PR solving?

  • 3 1 typos in README.md and thrift-gen/zipkincore/ttypes.go

Signed-off-by: Marc Bramaud <m.duboucheron@gmail.com>
@codecov
Copy link

codecov bot commented Dec 13, 2019

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #475   +/-   ##
======================================
  Coverage    88.1%   88.1%           
======================================
  Files          59      59           
  Lines        3507    3507           
======================================
  Hits         3090    3090           
  Misses        305     305           
  Partials      112     112

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 3585cc5...83c2f22. Read the comment docs.

@@ -741,7 +741,7 @@ func (p *BinaryAnnotation) String() string {
// precise measurement decoupled from problems of clocks, such as skew or NTP
// updates causing time to move backwards.
//
// For compatibilty with instrumentation that precede this field, collectors
// For compatibiltiy with instrumentation that precede this field, collectors
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please do not make changes in the auto-generated files

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah, my bad. Can you point me to the source so I can make the change there?

Signed-off-by: Marc Bramaud <m.duboucheron@gmail.com>
@sircelsius
Copy link
Contributor Author

I reverted the changes to generated files and only kept the one to README.

If you could point me to the source of the generated files so I can fix the typo there it would be amazing, I'll include that change in another PR.

@yurishkuro
Copy link
Member

Thanks. The source is most likely in the jaeger-idl repo which is included here as a submodule.

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.

None yet

2 participants