Skip to content

Conversation

@codeboten
Copy link
Contributor

@codeboten codeboten commented Apr 17, 2024

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes #2428

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

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

Signed-off-by: Alex Boten <[email protected]>
@codeboten codeboten marked this pull request as ready for review April 23, 2024 20:46
@codeboten codeboten requested a review from a team April 23, 2024 20:46
@xrmx
Copy link
Contributor

xrmx commented Apr 24, 2024

Please also fix formatting with updated black

Signed-off-by: Alex Boten <[email protected]>
Signed-off-by: Alex Boten <[email protected]>
@codeboten
Copy link
Contributor Author

@xrmx ran black + isort and updated the license

Signed-off-by: Alex Boten <[email protected]>
Signed-off-by: Alex Boten <[email protected]>
@ocelotl ocelotl merged commit 5375acf into open-telemetry:main Apr 25, 2024
@codeboten codeboten deleted the codeboten/baggage-processor branch April 25, 2024 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Request for new component: Baggage Span Processor

3 participants