Skip to content

Remove CONTRIBUTING guidelines for moved GenAI instrumentations - #4924

Merged
aabmass merged 2 commits into
open-telemetry:mainfrom
tammy-baylis-swi:update-contributing-genai
Aug 6, 2026
Merged

Remove CONTRIBUTING guidelines for moved GenAI instrumentations#4924
aabmass merged 2 commits into
open-telemetry:mainfrom
tammy-baylis-swi:update-contributing-genai

Conversation

@tammy-baylis-swi

Copy link
Copy Markdown
Contributor

Description

Python GenAI framework instrumentations have been moved to https://github.com/open-telemetry/opentelemetry-python-genai/ and we are deprecating the duplicates that are in this repo: #4918. This CONTRIBUTING.md update removes the outdated guidelines and points to the modern repo. IMO the modern guidelines are nice and simple, and I don't want to port over what was here. But LMK your thoughts!

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Not tested.

Does This PR Require a Core Repo Change?

  • Yes. - Link to PR:
  • 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

Comment thread CONTRIBUTING.md
@tammy-baylis-swi tammy-baylis-swi added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Aug 4, 2026
@tammy-baylis-swi
tammy-baylis-swi marked this pull request as ready for review August 4, 2026 21:39
@tammy-baylis-swi
tammy-baylis-swi requested a review from a team as a code owner August 4, 2026 21:39
@tammy-baylis-swi tammy-baylis-swi moved this to Ready for review in Python PR digest Aug 4, 2026
@tammy-baylis-swi tammy-baylis-swi moved this from Ready for review to Easy to review / merge / close in Python PR digest Aug 4, 2026

@lmolkova lmolkova left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I have a related PR here #4918 which removes a few other things, would appreciate some eyes on it, but we should probably merge this and then I'll resolve conflicts

Comment thread CONTRIBUTING.md
@aabmass
aabmass enabled auto-merge August 4, 2026 21:48
@aabmass aabmass moved this from Easy to review / merge / close to Approved PRs that need fixes in Python PR digest Aug 4, 2026
@aabmass
aabmass added this pull request to the merge queue Aug 6, 2026
Merged via the queue into open-telemetry:main with commit ac3673d Aug 6, 2026
777 checks passed
@github-project-automation github-project-automation Bot moved this from Approved PRs that need fixes to Done in Python PR digest Aug 6, 2026
@tammy-baylis-swi
tammy-baylis-swi deleted the update-contributing-genai branch August 7, 2026 17:39
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