[PR #140] [MERGED] QoL Changes #661

Closed
opened 2026-02-16 00:18:44 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/run-llama/llama_cloud_services/pull/140
Author: @logan-markewich
Created: 4/14/2024
Status: Merged
Merged: 4/14/2024
Merged by: @logan-markewich

Base: mainHead: logan/qol_fixes


📝 Commits (10+)

📊 Changes

29 files changed (+2203 additions, -2074 deletions)

View changed files

.github/workflows/build_package.yml (+48 -0)
.github/workflows/codeql.yml (+81 -0)
.github/workflows/lint.yml (+37 -0)
.github/workflows/publish_release.yml (+64 -0)
.github/workflows/unit_test.yml (+40 -0)
📝 .gitignore (+1 -1)
.pre-commit-config.yaml (+86 -0)
Makefile (+14 -0)
📝 README.md (+8 -5)
📝 examples/agents/demo_simple_openai_agent.ipynb (+22 -22)
📝 examples/demo_advanced.ipynb (+33 -28)
📝 examples/demo_advanced_astradb.ipynb (+42 -33)
📝 examples/demo_api.ipynb (+6 -8)
📝 examples/demo_astradb.ipynb (+21 -22)
📝 examples/demo_basic.ipynb (+8 -15)
📝 examples/demo_insurance.ipynb (+56 -80)
📝 examples/demo_json.ipynb (+38 -73)
📝 examples/demo_json_parsing.ipynb (+26 -24)
📝 examples/demo_languages.ipynb (+22 -59)
📝 examples/demo_mongodb.ipynb (+358 -425)

...and 9 more files

📄 Description

  • make loading many files more robust
    • added show_progress option
    • added error handling to avoid crashing when loading many files
    • use the actual status endpoint for checking job status, to avoid 404 logs/requests
  • Added CICD
    • linting
    • pre-commit
    • publishing
    • package building
    • tests (using token in GitHub secrets)
  • Added more tests
    • test that num_workers works
    • test that show_progress doesn't crash

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/run-llama/llama_cloud_services/pull/140 **Author:** [@logan-markewich](https://github.com/logan-markewich) **Created:** 4/14/2024 **Status:** ✅ Merged **Merged:** 4/14/2024 **Merged by:** [@logan-markewich](https://github.com/logan-markewich) **Base:** `main` ← **Head:** `logan/qol_fixes` --- ### 📝 Commits (10+) - [`a6bba17`](https://github.com/run-llama/llama_cloud_services/commit/a6bba17c613d81008b01401669fd10ba9cc6e2f2) add linting, make loading more robust - [`aef1855`](https://github.com/run-llama/llama_cloud_services/commit/aef185579e7ef8503ab76a8674b2e3bb710eec89) Add CICD - [`8d08d54`](https://github.com/run-llama/llama_cloud_services/commit/8d08d54caa8ab41a27a00b317afbd99ba865da50) add api key to env - [`2480d34`](https://github.com/run-llama/llama_cloud_services/commit/2480d34ca5505fa31a8f4cb6bce966bd0d207333) more tests - [`f0aa148`](https://github.com/run-llama/llama_cloud_services/commit/f0aa14853a109446d24775fe40894ad5e557fa1f) bump llama-index-core version - [`e3842dd`](https://github.com/run-llama/llama_cloud_services/commit/e3842dd7eddb822816200bde5ef7f1089a8ebfb0) ensure poetry lock works in CICD - [`1b60242`](https://github.com/run-llama/llama_cloud_services/commit/1b6024231341aa82363512f0d07939a101e37149) add makefile, whoops - [`66a0c5b`](https://github.com/run-llama/llama_cloud_services/commit/66a0c5ba45128c48c891aae382154f92f071dfaa) use default runner for unit tests - [`dc90d76`](https://github.com/run-llama/llama_cloud_services/commit/dc90d76da22979b8c50415cfe0abd63ed9fd7d49) proper testing install - [`5e72798`](https://github.com/run-llama/llama_cloud_services/commit/5e727988445f084499f27e2990aa29f56c86f9bb) jk we cant install 3.12 stil? ### 📊 Changes **29 files changed** (+2203 additions, -2074 deletions) <details> <summary>View changed files</summary> ➕ `.github/workflows/build_package.yml` (+48 -0) ➕ `.github/workflows/codeql.yml` (+81 -0) ➕ `.github/workflows/lint.yml` (+37 -0) ➕ `.github/workflows/publish_release.yml` (+64 -0) ➕ `.github/workflows/unit_test.yml` (+40 -0) 📝 `.gitignore` (+1 -1) ➕ `.pre-commit-config.yaml` (+86 -0) ➕ `Makefile` (+14 -0) 📝 `README.md` (+8 -5) 📝 `examples/agents/demo_simple_openai_agent.ipynb` (+22 -22) 📝 `examples/demo_advanced.ipynb` (+33 -28) 📝 `examples/demo_advanced_astradb.ipynb` (+42 -33) 📝 `examples/demo_api.ipynb` (+6 -8) 📝 `examples/demo_astradb.ipynb` (+21 -22) 📝 `examples/demo_basic.ipynb` (+8 -15) 📝 `examples/demo_insurance.ipynb` (+56 -80) 📝 `examples/demo_json.ipynb` (+38 -73) 📝 `examples/demo_json_parsing.ipynb` (+26 -24) 📝 `examples/demo_languages.ipynb` (+22 -59) 📝 `examples/demo_mongodb.ipynb` (+358 -425) _...and 9 more files_ </details> ### 📄 Description - make loading many files more robust - added `show_progress` option - added error handling to avoid crashing when loading many files - use the actual status endpoint for checking job status, to avoid 404 logs/requests - Added CICD - linting - pre-commit - publishing - package building - tests (using token in GitHub secrets) - Added more tests - test that num_workers works - test that show_progress doesn't crash --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 00:18:44 -05:00
yindo closed this issue 2026-02-16 00:18:44 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: run-llama/llama_cloud_services#661