python3Packages.langchain*: Move dependencies of langchain-core to a common update script #342114
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sarahec
force-pushed
the
langchain-common-update
branch
2 times, most recently
from
6f1a597
to
dffa0a2
Compare
sarahec
force-pushed
the
langchain-common-update
branch
2 times, most recently
from
847c20b
to
c14f150
Compare
13 tasks
|
Result of |
ofborg
bot
added
10.rebuild-darwin: 0
wegank
added
the
12.approvals: 1
natsukium
reviewed
Sep 17, 2024
sarahec
force-pushed
the
langchain-common-update
branch
from
c14f150
to
fc5c940
Compare
wegank
removed
the
12.approvals: 1
sarahec
added
the
12.approvals: 1
sarahec
force-pushed
the
langchain-common-update
branch
from
fc5c940
to
2339990
Compare
wegank
removed
the
12.approvals: 1
…common update script
sarahec
force-pushed
the
langchain-common-update
branch
from
2339990
to
1b414a4
Compare
|
Result of |
|
Result of |
|
Result of |
|
Result of |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of changes
Set all dependencies of
langchain-coreto use a common update script (that lists all of them) . Rationale: a core update tends to triger updates across the whole set. (This also provides guidance to anyone who updates langchain-core manually to check the rest of the set.)Note: This shouldn't trigger any rebuilds.
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.