Skip to content

Removed unused span manager from langchain#190

Merged
lmolkova merged 2 commits into
open-telemetry:mainfrom
rads-1996:remove-unused-files-langchain
Jul 3, 2026
Merged

Removed unused span manager from langchain#190
lmolkova merged 2 commits into
open-telemetry:mainfrom
rads-1996:remove-unused-files-langchain

Conversation

@rads-1996

Copy link
Copy Markdown
Contributor

Removes the unused span_manager.py from the langchain instrumentation.

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 removes an unused internal span-management module from the LangChain GenAI instrumentation package, reducing dead code and avoiding maintenance of logic that is no longer referenced.

Changes:

  • Deleted span_manager.py, which contained the unused _SpanManager implementation.

@eternalcuriouslearner

Copy link
Copy Markdown
Contributor

@wrisa Can you please take a look into this?

@eternalcuriouslearner

Copy link
Copy Markdown
Contributor

@rads-1996 Thanks for your contribution, can you please add a changelog in towncrier format?

@rads-1996

Copy link
Copy Markdown
Contributor Author

@rads-1996 Thanks for your contribution, can you please add a changelog in towncrier format?

@eternalcuriouslearner Added the changelog.

@lmolkova lmolkova added this pull request to the merge queue Jul 3, 2026
Merged via the queue into open-telemetry:main with commit e05aef4 Jul 3, 2026
85 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants