New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: added support for Opentelemetry 0.18 #1234
Merged
Merged
Changes from 1 commit
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
cf0217d
fix: added support for Opentelemetry 0.18
tapico-weyert c01a210
style: remove unused import declaration
tapico-weyert 0de436e
refactor: removed the `OpenTelemetryTracer`-class and just expose cre…
tapico-weyert c36c9f5
test: updated the span related tests
tapico-weyert e790c23
test: updated the tests
tapico-weyert d96d2d9
test: updated the tests
tapico-weyert 92b1a4c
refactor: move the tracer out of the `createSpan`-function
tapico-weyert b884b21
Merge branch 'master' into remove-tracing-dependency
feywind 8b8d97e
Merge branch 'master' into remove-tracing-dependency
feywind 560cdb2
Merge branch 'master' into remove-tracing-dependency
feywind 1f6f7e2
fix: get the unit test for publisher opentelemetry working again
feywind 0035983
test: fix the subscriber unit tests for opentelemetry
tapico-weyert 3d33208
chore: remove describe.only
tapico-weyert 364de28
chore: latest changes
tapico-weyert 93eaaa1
test: updated the tests
tapico-weyert 31b8371
test: improve the tests
tapico-weyert 6159fb3
chore: maybe its need to be Google LLC?
tapico-weyert f26d90e
feat: add messaging attributes to otel spans
tapico-weyert 2cbccb9
fix: include package.json version as instrumentation version
tapico-weyert 93760c3
docs: update the opentelemetry example code
tapico-weyert 54deb7c
style: remove unused code
tapico-weyert 99adf22
fix: remove peer name for now
tapico-weyert cff5314
style: improve explanation regarding otel trace provider
tapico-weyert ae3e959
chore: update the opentelemetry versios in the samples `package.json`
tapico-weyert 2e0095b
test: remove extranous logging in the opentelemetry example app
tapico-weyert 78f31e4
test: remove the check for `traceId` that doesn't get outputted
tapico-weyert a73c74a
fix: change reference to `package.json` one level lower
tapico-weyert e42ac63
fix: remove unnecessary messaging span attribute and added comments
tapico-weyert 0c54251
style: use shiny new syntax for undefined check in `subscriber`-class
tapico-weyert d0f0498
fix: match the span name with the operation kind in `subscriber`-class
tapico-weyert 6c0ca40
chore: improved code based on PR feedback
tapico-weyert 818013a
fix:
tapico-weyert 2cbaa28
fix: only include `googclient_OpenTelemetrySpanContext`-attribute
tapico-weyert 0c2c76f
fix: change the way `package.json` is being imported
tapico-weyert 8d600ca
Merge branch 'master' into remove-tracing-dependency
feywind 4b93dd1
fix: revert bad merge that removed tracing on devDependencies
feywind afe0529
style: remove unnecessary comment
weyert fa66633
style: remove unused imports
tapico-weyert 7a628e9
build: downgrade @sinonjs/fake-timers to v6
tapico-weyert c1b66dd
style: cleanup type definition
weyert c081f6b
build: remove `package.json` after compilation step
tapico-weyert b42344d
chore: merge from main
feywind 9339baf
build: put back package.json in the build
feywind 14de625
build: explicitly add package.json to the npm pack build
feywind 253b56b
fix: revert "fix: change the way `package.json` is being imported"
feywind 4db9ba5
fix: revert "build: explicitly add package.json to the npm pack build"
feywind 4f360e4
Merge branch 'master' into remove-tracing-dependency
feywind File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you would need to revert the change from
require
toimport
approach for loadingpackage.json
. I don't think removing this is enough