Skip to content

Add skills to help with instrumentation migration#143

Open
lmolkova wants to merge 6 commits into
open-telemetry:mainfrom
lmolkova:add-migration-skills-2
Open

Add skills to help with instrumentation migration#143
lmolkova wants to merge 6 commits into
open-telemetry:mainfrom
lmolkova:add-migration-skills-2

Conversation

@lmolkova

Copy link
Copy Markdown
Member
  • migrate-from-openinference — migrate an openinference-instrumentation-*
    package into this repo as an OTel GenAI package, or augment an existing
    package with the coverage OpenInference adds on top.
  • review-migration — review a ported or augmented package against its
    upstream implementation and write MIGRATION_REPORT.md.
  • write-conformance-tests — author conformance scenarios and the
    test_conformance.py runner for an instrumentation package.

@lmolkova lmolkova marked this pull request as ready for review June 16, 2026 16:18
@lmolkova lmolkova requested a review from a team as a code owner June 16, 2026 16:18
Copilot AI review requested due to automatic review settings June 16, 2026 16:18
@lmolkova lmolkova added the Skip Changelog PR does not require a changelog entry label Jun 16, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds three repository “skills” under .github/skills/ to streamline common instrumentation migration workflows (migration from OpenInference, migration review reports, and authoring conformance tests), and documents their intended usage for contributors.

Changes:

  • Documented available skills and when to trigger them in CONTRIBUTING.md.
  • Added gitignore rule for generated MIGRATION_REPORT.md outputs.
  • Introduced three new skills: migrate-from-openinference, review-migration, and write-conformance-tests.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
CONTRIBUTING.md Adds a “Skills” section describing the new automation skills and when to use them.
.gitignore Ignores generated MIGRATION_REPORT.md files.
.github/skills/write-conformance-tests/SKILL.md New skill: guidance for writing GenAI conformance scenarios and the runner.
.github/skills/review-migration/SKILL.md New skill: guidance for reviewing migrations and generating MIGRATION_REPORT.md.
.github/skills/migrate-from-openinference/SKILL.md New skill: step-by-step workflow for migrating/augmenting OpenInference instrumentations into this repo.

Comment thread .github/skills/migrate-from-openinference/SKILL.md Outdated
Comment thread .github/skills/review-migration/SKILL.md Outdated
Comment thread .github/skills/migrate-from-openinference/SKILL.md
Comment thread .github/skills/migrate-from-openinference/SKILL.md Outdated
Comment thread .github/skills/migrate-from-openinference/SKILL.md
Comment thread CONTRIBUTING.md
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
@lmolkova lmolkova force-pushed the add-migration-skills-2 branch 2 times, most recently from e5b9e1c to ab155f2 Compare June 17, 2026 01:21
@lmolkova lmolkova force-pushed the add-migration-skills-2 branch from 30b11bd to 9a035d7 Compare June 17, 2026 01:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PR does not require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants