-
Notifications
You must be signed in to change notification settings - Fork 5.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The llamaindex cli package #10723
The llamaindex cli package #10723
Conversation
version = "0.1.0" | ||
|
||
[tool.poetry.dependencies] | ||
python = ">=3.8.1,<3.12" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
btw, I've noticed we have <3.12
in all our packages. We might want to consider making this <4.0.0
🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It can be done ofc, but it might be a bit more involved then just changing pyproject.toml's :)
d491069
to
f970fa6
Compare
* llamaindex cli package * BUILD to pass tests * generate pants BUILD files * add skip tests * use llamaindex-cli name * CHANGELOG
* llamaindex cli package * BUILD to pass tests * generate pants BUILD files * add skip tests * use llamaindex-cli name * CHANGELOG
Description
Part 1: Ripping out CLI from
llama-index-core
llama-index-cli
llamaindex
rather thanllamaindex-cli
. This is the case cause core currently occupies that and its still not ripped out from core.For Part 2:
llama-index
python packagellamaindex-cli
fromllama-index-core
Fixes # (issue)
Type of Change
Please delete options that are not relevant.