Skip to content

Conversation

@xrmx
Copy link
Member

@xrmx xrmx commented Nov 10, 2025

What does this pull request do?

Update the migration guide with more option that already have counterparts in OpenTelemetry Python.
Also use environment variable instead of option if that's the case.

Related issues

@xrmx xrmx requested a review from a team as a code owner November 10, 2025 16:03
@xrmx xrmx force-pushed the update-migrate-guide branch from bc4b3d0 to 67c3dad Compare November 10, 2025 16:04
@xrmx xrmx force-pushed the update-migrate-guide branch from 5f93ec5 to 999cc09 Compare November 10, 2025 16:25
@github-actions
Copy link

github-actions bot commented Nov 10, 2025

🔍 Preview links for changed docs

@xrmx xrmx changed the title Update migrate guide docs: update migration guide Nov 10, 2025
Copy link
Contributor

@theletterf theletterf left a comment

Choose a reason for hiding this comment

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

Just some nits

@xrmx xrmx requested a review from theletterf November 11, 2025 09:39
@xrmx
Copy link
Member Author

xrmx commented Nov 11, 2025

I think I may have missed some suggestions 🤔

@xrmx
Copy link
Member Author

xrmx commented Nov 11, 2025

I think I may have missed some suggestions 🤔

Added the 3 missing suggestions

Copy link
Contributor

@theletterf theletterf left a comment

Choose a reason for hiding this comment

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

🎊

@xrmx xrmx merged commit 504f5c0 into main Nov 11, 2025
16 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.

3 participants