Skip to content

fix(langchain-classic): pass default to config.getoption#34034

Merged
Mason Daugherty (mdrxy) merged 5 commits intolangchain-ai:masterfrom
sarahec:classic-config-getoption-default
Dec 27, 2025
Merged

fix(langchain-classic): pass default to config.getoption#34034
Mason Daugherty (mdrxy) merged 5 commits intolangchain-ai:masterfrom
sarahec:classic-config-getoption-default

Conversation

@sarahec
Copy link
Copy Markdown
Contributor

@sarahec Sarah Clark (sarahec) commented Nov 19, 2025

Fixes #34033

@github-actions github-actions bot added langchain-classic `langchain-classic` package issues & PRs fix For PRs that implement a fix labels Nov 19, 2025
@eyurtsev Eugene Yurtsev (eyurtsev) changed the title fix(langchain-classic): Pass default to config.getoption fix(langchain-classic): pass default to config.getoption Nov 19, 2025
@github-actions github-actions bot added fix For PRs that implement a fix and removed fix For PRs that implement a fix labels Nov 19, 2025
@mdrxy
Copy link
Copy Markdown
Member

Should we also extend this to libs/langchain_v1/tests/unit_tests/conftest.py?

@github-actions github-actions bot added fix For PRs that implement a fix and removed fix For PRs that implement a fix labels Dec 11, 2025
@mdrxy
Copy link
Copy Markdown
Member

Sarah Clark (@sarahec) quickly following up

@mdrxy Mason Daugherty (mdrxy) added the waiting-on-author Issue or PR is paused, waiting to hear from author in order to proceed label Dec 27, 2025
@sarahec
Copy link
Copy Markdown
Contributor Author

Sarah Clark (sarahec) commented Dec 27, 2025

Should we also extend this to libs/langchain_v1/tests/unit_tests/conftest.py?

I'm sorry, I missed seeing that. Yes, I can make the edit.

UPDATE: Done.

@github-actions github-actions bot added the langchain `langchain` package issues & PRs label Dec 27, 2025
@mdrxy Mason Daugherty (mdrxy) merged commit 10de0a5 into langchain-ai:master Dec 27, 2025
47 checks passed
@mdrxy Mason Daugherty (mdrxy) removed the waiting-on-author Issue or PR is paused, waiting to hear from author in order to proceed label Dec 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external fix For PRs that implement a fix langchain `langchain` package issues & PRs langchain-classic `langchain-classic` package issues & PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

langchain-classic: pytest fails with a ValueError when building on nixpkgs

2 participants