Skip to content
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

Remove stale packages from main #17235

Merged
merged 3 commits into from
Dec 10, 2024
Merged

Conversation

nerdai
Copy link
Contributor

@nerdai nerdai commented Dec 10, 2024

Description

Dec 10 review of integration packages found the following 20 packages to be stale:

llama-index-embeddings-huggingface-itrex | 0.0006430868167
llama-index-packs-redis-ingestion-pipeline | 0.0006430868167
llama-index-packs-subdoc-summary | 0.0006430868167
llama-index-packs-finchat | 0.0006430868167
llama-index-readers-snscrape-twitter | 0.000667495131
llama-index-llms-solar | 0.0006714967112
llama-index-readers-readme | 0.0006720488261
llama-index-llms-unify | 0.0006723174281
llama-index-packs-docugami-kg-rag | 0.0006738850004
llama-index-packs-searchain | 0.0006739676584
llama-index-readers-openapi | 0.0006758560113
llama-index-readers-youtube-metadata | 0.0006768877976
llama-index-tools-passio-nutrition-ai | 0.0006776269773
llama-index-readers-feishu-wiki | 0.0006791342778
llama-index-embeddings-octoai | 0.0006798481852
llama-index-readers-clickhouse | 0.0006801570384
llama-index-readers-azure-devops | 0.0006826668916

Also moved to stale:

llama-index-vector-stores-chatgpt-plugin
llama-index-vector-stores-metal
llama-index-packs-vanna

Companion PR to move these packages to stale_packages branch: #17234

@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Dec 10, 2024
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@logan-markewich
Copy link
Collaborator

Can we do a double pass and remove any references to these packages in our docs?

@nerdai
Copy link
Contributor Author

nerdai commented Dec 10, 2024

Can we do a double pass and remove any references to these packages in our docs?

Good call! Will do that now.

@nerdai nerdai force-pushed the nerdai/remove-stale-packages-from-main branch from 87fa809 to 2a5d250 Compare December 10, 2024 18:29
@nerdai
Copy link
Contributor Author

nerdai commented Dec 10, 2024

Alright, that should take care of everything in the docs. It looks like one of the stale packages, namely llama-index-llms-solar was recently modified in #17231. But it looks like it was only to rip out mention of llama-index-legacy, and so it should still be deemed as stale.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 10, 2024
@nerdai nerdai merged commit 37832ff into main Dec 10, 2024
11 checks passed
@nerdai nerdai deleted the nerdai/remove-stale-packages-from-main branch December 10, 2024 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants