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

Warning for mismatching env variables #332

Merged
merged 7 commits into from
Dec 24, 2018

Conversation

yepninja
Copy link
Contributor

Warning for a minor release.

Signed-off-by: Yevgeny Petukhov <[email protected]>
Signed-off-by: Yevgeny Petukhov <[email protected]>
src/configuration_env.js Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Dec 22, 2018

Codecov Report

Merging #332 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #332      +/-   ##
==========================================
+ Coverage   98.95%   98.95%   +<.01%     
==========================================
  Files          43       43              
  Lines        1718     1725       +7     
  Branches      335      336       +1     
==========================================
+ Hits         1700     1707       +7     
  Misses         18       18
Impacted Files Coverage Δ
src/configuration_env.js 100% <100%> (ø) ⬆️

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 e71ef4b...aee87a5. Read the comment docs.

Copy link
Member

@yurishkuro yurishkuro left a comment

Choose a reason for hiding this comment

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

does this PR finish the ticket #321 ?

src/configuration_env.js Outdated Show resolved Hide resolved
src/configuration_env.js Show resolved Hide resolved
yurishkuro
yurishkuro previously approved these changes Dec 23, 2018
Copy link
Member

@yurishkuro yurishkuro left a comment

Choose a reason for hiding this comment

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

Thanks!

@yurishkuro yurishkuro dismissed their stale review December 23, 2018 19:27

build is failing

@yepninja
Copy link
Contributor Author

Opps. Almost there is no habit to code without eslint.
The build is fixed!

Copy link
Member

@yurishkuro yurishkuro left a comment

Choose a reason for hiding this comment

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

Thanks!

@yurishkuro
Copy link
Member

coming back to my earlier question - does this PR finish the ticket #321 ?

@yepninja
Copy link
Contributor Author

Nope. That issue is related to PR #330

@yurishkuro yurishkuro merged commit d244bc0 into jaegertracing:master Dec 24, 2018
Iuriy-Budnikov pushed a commit to agile-pm/jaeger-client-node that referenced this pull request Sep 25, 2021
* fix: dont trace ourselves

closes jaegertracing#332

Signed-off-by: Olivier Albertini <[email protected]>

* fix: add mayurkale22 recommendations

Signed-off-by: Olivier Albertini <[email protected]>
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.

2 participants