Skip to content

Conversation

emdneto
Copy link
Member

@emdneto emdneto commented Dec 11, 2024

Description

Improve contributing guide to make explicit how to support new versions of instrumentation packages

@emdneto emdneto requested a review from a team as a code owner December 11, 2024 00:49
@emdneto emdneto added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Dec 11, 2024
Copy link
Contributor

@codefromthecrypt codefromthecrypt left a comment

Choose a reason for hiding this comment

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

Looks good, I have some suggestions to clarify the location of where commands should run. Also here and everywhere else in this file, we should switch from backticking tox.ini to a link to it. This helps markdown link checker ensure there's not a typo there, also, readers can click it!

@xrmx xrmx merged commit 1ddba1b into open-telemetry:main Dec 18, 2024
573 checks passed
xrmx added a commit to xrmx/opentelemetry-python-contrib that referenced this pull request Jan 24, 2025
…pen-telemetry#3090)

* contributing: add guidance for update package versions support

Signed-off-by: emdneto <[email protected]>

* update the guide

Signed-off-by: emdneto <[email protected]>

* Update CONTRIBUTING.md

* Update CONTRIBUTING.md

Co-authored-by: Riccardo Magliocchetti <[email protected]>

* Update CONTRIBUTING.md

Co-authored-by: Riccardo Magliocchetti <[email protected]>

* Update CONTRIBUTING.md to add docs instructions

* Update CONTRIBUTING.md

* Update CONTRIBUTING.md

---------

Signed-off-by: emdneto <[email protected]>
Co-authored-by: Riccardo Magliocchetti <[email protected]>
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

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants