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.

Sign up for GitHub

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

Jump to bottom

docs: grammar #3167

Merged
merged 3 commits into from
Sep 16, 2022
Merged

docs: grammar #3167

merged 3 commits into from
Sep 16, 2022

Conversation

ttd2089
Copy link
Contributor

@ttd2089 ttd2089 commented Sep 13, 2022

No description provided.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Sep 13, 2022 •
edited

CLA Signed

The committers listed above are authorized under a signed CLA.

@MrAlias MrAlias added the blocked:CLA Waiting on CLA to be signed before progress can be made label Sep 13, 2022
@MrAlias
Copy link
Contributor

MrAlias commented Sep 13, 2022

Thanks for the fix. Please sign the CLA so the change can be accepted.

@MrAlias MrAlias added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Sep 13, 2022
@Aneurysm9 Aneurysm9 removed the blocked:CLA Waiting on CLA to be signed before progress can be made label Sep 13, 2022
@MrAlias MrAlias added the documentation Provides helpful information label Sep 13, 2022
@codecov
Copy link

codecov bot commented Sep 13, 2022 •
edited

Codecov Report

Merging #3167 (ed4c698) into main (8af1d31) will increase coverage by 0.0%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #3167   +/-   ##
=====================================
  Coverage   77.3%   77.3%           
=====================================
  Files        158     158           
  Lines      11082   11082           
=====================================
+ Hits        8575    8577    +2     
+ Misses      2310    2308    -2     
  Partials     197     197
Impacted Files Coverage Δ
sdk/trace/batch_span_processor.go 81.9% <0.0%> (+0.8%) ⬆️

@MrAlias MrAlias merged commit e1a1f07 into open-telemetry:main Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Aneurysm9 Aneurysm9 Aneurysm9 approved these changes

@MrAlias MrAlias MrAlias approved these changes

@XSAM XSAM XSAM approved these changes

@hanyuancheung hanyuancheung hanyuancheung approved these changes

@jmacd jmacd Awaiting requested review from jmacd

@evantorrie evantorrie Awaiting requested review from evantorrie

@dashpole dashpole Awaiting requested review from dashpole dashpole is a code owner

@MadVikingGod MadVikingGod Awaiting requested review from MadVikingGod MadVikingGod is a code owner

@pellared pellared Awaiting requested review from pellared pellared is a code owner

@dmathieu dmathieu Awaiting requested review from dmathieu dmathieu is a code owner

Assignees
No one assigned
Labels
documentation Provides helpful information Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@ttd2089 @MrAlias @Aneurysm9 @XSAM @hanyuancheung