Skip to content

Conversation

@ecausarano
Copy link
Contributor

@ecausarano ecausarano commented Apr 2, 2025

Description:

removed a predicate marked Deprecated.

Existing Issue(s):

resolves #1827

Testing:

checked in the IDE if the code is used anywhere.

Documentation:

none, minimal code change removing a deprecated predicate.

Outstanding items:

none

@ecausarano ecausarano requested a review from a team April 2, 2025 15:39
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Apr 2, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: ecausarano / name: Edoardo Causarano (4a01347)

@breedx-splk breedx-splk enabled auto-merge (squash) April 2, 2025 15:45
auto-merge was automatically disabled April 2, 2025 15:46

Head branch was pushed to by a user without write access

@ecausarano ecausarano force-pushed the 1827-Remove-deprecated-constant-in-BaggageSpanProcessor branch from fbf69b5 to 4a01347 Compare April 2, 2025 15:46
Copy link
Member

@MikeGoldsmith MikeGoldsmith left a comment

Choose a reason for hiding this comment

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

Removing this is fine as long as we indicate it's been removed in the release notes as a user could have used it directly.

@trask trask merged commit d5790eb into open-telemetry:main Apr 3, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[contribfest] Remove deprecated constant in BaggageSpanProcessor

4 participants