Skip to content

chore: Exclude test packages from setuptools#40

Merged
Abhijeet Prasad (AbhiPrasad) merged 3 commits intomainfrom
langchain-tests-include
Mar 4, 2026
Merged

chore: Exclude test packages from setuptools#40
Abhijeet Prasad (AbhiPrasad) merged 3 commits intomainfrom
langchain-tests-include

Conversation

@colinbennettbrain
Copy link
Copy Markdown
Contributor

Fix braintrust-langchain shipping tests/ as an installed top-level package, which breaks pytest discovery in downstream projects.

- Implemented _on_ending method in AISpanProcessor and BraintrustSpanProcessor to forward pre-end hooks.
- Added a helper function _forward_on_ending to handle compatibility with different OpenTelemetry SDK versions.
- Updated tests to verify the presence and functionality of the new _on_ending method.
@AbhiPrasad Abhijeet Prasad (AbhiPrasad) changed the title Exclude test packages from setuptools chore: Exclude test packages from setuptools Mar 4, 2026
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.

Thanks!

@AbhiPrasad Abhijeet Prasad (AbhiPrasad) merged commit f60a2d3 into main Mar 4, 2026
33 checks passed
@AbhiPrasad Abhijeet Prasad (AbhiPrasad) deleted the langchain-tests-include branch March 6, 2026 05:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants