Skip to content

remove the ScopeWrapper workaround#9384

Merged
bvahdat merged 1 commit intomainfrom
remove-ScopeWrapper
Feb 22, 2023
Merged

remove the ScopeWrapper workaround#9384
bvahdat merged 1 commit intomainfrom
remove-ScopeWrapper

Conversation

@bvahdat
Copy link
Member

@bvahdat bvahdat commented Feb 19, 2023

Description

Target

  • I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

  • If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

  • I checked that each commit in the pull request has a meaningful subject line and body.
  • I formatted the code using mvn -Pformat,fastinstall install && mvn -Psourcecheck

As open-telemetry/opentelemetry-java#5055 seems to be fixed.

@github-actions
Copy link
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟

⚠️ Please note that the changes on this PR may be tested automatically.

If necessary Apache Camel Committers may access logs and test results in the job summaries!

@bvahdat
Copy link
Member Author

bvahdat commented Feb 19, 2023

@lmolkova would you mind to review this please?

@github-actions
Copy link
Contributor

Components tested:

Total Tested Failed ❌ Passed ✅
1 1 0 1

Copy link

@lmolkova lmolkova left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@bvahdat bvahdat merged commit d8a01e7 into main Feb 22, 2023
@bvahdat bvahdat deleted the remove-ScopeWrapper branch February 22, 2023 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants