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 comments #4553

Merged
merged 3 commits into from Sep 26, 2023
Merged

Conversation

CharlieTLe
Copy link
Contributor

No description provided.

@dmathieu dmathieu added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Sep 25, 2023
sdk/metric/meter_test.go Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Sep 26, 2023

Codecov Report

Merging #4553 (4df5485) into main (2a8fdda) will increase coverage by 0.0%.
The diff coverage is n/a.

❗ Current head 4df5485 differs from pull request most recent head b577054. Consider uploading reports for the commit b577054 to get more accurate results

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #4553   +/-   ##
=====================================
  Coverage   81.3%   81.3%           
=====================================
  Files        220     220           
  Lines      17679   17679           
=====================================
+ Hits       14384   14386    +2     
+ Misses      2994    2992    -2     
  Partials     301     301           
Files Coverage Δ
sdk/metric/reader.go 100.0% <ø> (ø)

... and 1 file with indirect coverage changes

@pellared pellared changed the title Fix minor typos in comments and test cases Fix typos in comments Sep 26, 2023
@pellared pellared merged commit 612208d into open-telemetry:main Sep 26, 2023
22 checks passed
@MrAlias MrAlias added this to the v1.19.0/v0.42.0 milestone Sep 28, 2023
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