Skip to content

Conversation

@vasantteja
Copy link
Contributor

Description

This PR has been largely influenced from this PR. Please take a look and provide me feedback if I need to correct or refine it further.

Part of #3949

P.S: LLM assistance was used for clarifying concepts and writing this boilerplate.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

This is a simple boilerplate and all the relevant tests I added around anthrophic sdk instantiation are passing.

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

@vasantteja vasantteja removed their assignment Nov 27, 2025
@xrmx xrmx moved this to Ready for review in @xrmx's Python PR digest Nov 27, 2025
@vasantteja vasantteja removed their assignment Nov 29, 2025
Copy link
Member

Choose a reason for hiding this comment

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

I think we can remove this file for now

build-backend = "hatchling.build"

[project]
name = "opentelemetry-instrumentation-anthropic"
Copy link
Member

Choose a reason for hiding this comment

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

This package name is already taken by OpenLLMetry: https://pypi.org/project/opentelemetry-instrumentation-anthropic/

Have you been in touch with them about sharing the package with OTel? If not, the approach we've been taking is adding -v2

"opentelemetry-instrumentation-openai-v2": "",
"opentelemetry-instrumentation-vertexai": ">=2.0b0",
"opentelemetry-instrumentation-google-genai": "",
"opentelemetry-instrumentation-anthropic": "",
Copy link
Member

Choose a reason for hiding this comment

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

Can you add it to packages_to_exclude array above until we have some releases?

# See the License for the specific language governing permissions and
# limitations under the License.

__version__ = "2.1b0.dev"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
__version__ = "2.1b0.dev"
__version__ = "2.0b0.dev"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready for review

Development

Successfully merging this pull request may close these issues.

7 participants