Skip to content
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 typos in resource.go. #2201

Merged
merged 3 commits into from
Aug 30, 2021
Merged

Fix typos in resource.go. #2201

merged 3 commits into from
Aug 30, 2021

Conversation

ronilichtman
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Aug 24, 2021

Codecov Report

Merging #2201 (3fdbd1a) into main (abf6afe) will decrease coverage by 0.0%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #2201     +/-   ##
=======================================
- Coverage   72.9%   72.8%   -0.1%     
=======================================
  Files        176     176             
  Lines      12161   12161             
=======================================
- Hits        8867    8865      -2     
- Misses      3055    3057      +2     
  Partials     239     239             
Impacted Files Coverage Δ
sdk/resource/resource.go 74.1% <ø> (ø)
exporters/jaeger/jaeger.go 93.4% <0.0%> (-0.9%) ⬇️

@@ -43,6 +43,7 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
- The `"go.opentelemetry.io/otel/attribute".Any` function now supports `int32` values. (#2169)
- Multiple calls to `"go.opentelemetry.io/otel/sdk/metric/controller/basic".WithResource()` are handled correctly, and when no resources are provided `"go.opentelemetry.io/otel/sdk/resource".Default()` is used. (#2120)
- The `WithoutTimestamps` option for the `go.opentelemetry.io/otel/exporters/stdout/stdouttrace` exporter causes the exporter to correctly ommit timestamps. (#2195)
- Fixed typos in resources.go. (#2201)
Copy link
Member

Choose a reason for hiding this comment

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

No need to have a changelog entry for this as it just fixes typos in docs and doesn't change anything for users.

@Aneurysm9 Aneurysm9 added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Aug 24, 2021
@Aneurysm9 Aneurysm9 merged commit c0c5ef6 into open-telemetry:main Aug 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants