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

Minor refactoring #1169

Merged
merged 6 commits into from
Dec 3, 2022
Merged

Minor refactoring #1169

merged 6 commits into from
Dec 3, 2022

Conversation

artemrys
Copy link
Contributor

Description

This PR refactors couple of existing instrumentation libraries and fixes couple of typos.

Type of change

  • Refactoring

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Unit tests

Does This PR Require a Core Repo Change?

  • No.

Checklist:

See contributing.md for styleguide, changelog guidelines, and more.

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jun 28, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

@artemrys artemrys closed this Oct 17, 2022
@artemrys artemrys reopened this Oct 17, 2022
@artemrys artemrys closed this Oct 17, 2022
@artemrys artemrys reopened this Oct 17, 2022
@artemrys artemrys closed this Nov 6, 2022
@artemrys artemrys reopened this Nov 6, 2022
@artemrys artemrys marked this pull request as ready for review November 6, 2022 10:43
@artemrys artemrys requested a review from a team November 6, 2022 10:43
@ocelotl ocelotl added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Nov 18, 2022
@ocelotl ocelotl enabled auto-merge (squash) November 18, 2022 14:44
@ocelotl ocelotl merged commit 155fc46 into open-telemetry:main Dec 3, 2022
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.

3 participants