[PR #873] [MERGED] Add timeout arg to github repo reader #887

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

📋 Pull Request Information

Original PR: https://github.com/run-llama/llama-hub/pull/873
Author: @rwood-97
Created: 1/18/2024
Status: Merged
Merged: 1/19/2024
Merged by: @nerdai

Base: mainHead: github-timeout


📝 Commits (2)

📊 Changes

3 files changed (+61 additions, -12 deletions)

View changed files

📝 llama_hub/github_repo/README.md (+2 -0)
📝 llama_hub/github_repo/base.py (+10 -3)
📝 llama_hub/github_repo/github_client.py (+49 -9)

📄 Description

Description

This PR adds a timeout arg to the GithubRepositoryReader to allow users to specify a timeout when making requests.
Fixes #846

Type of Change

Please delete options that are not relevant.

  • Bug fix / Smaller change

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Added new unit/integration tests
  • I stared at the code and made sure it makes sense

Suggested Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I ran make format; make lint to appease the lint gods

🔄 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-hub/pull/873 **Author:** [@rwood-97](https://github.com/rwood-97) **Created:** 1/18/2024 **Status:** ✅ Merged **Merged:** 1/19/2024 **Merged by:** [@nerdai](https://github.com/nerdai) **Base:** `main` ← **Head:** `github-timeout` --- ### 📝 Commits (2) - [`f23c0cc`](https://github.com/run-llama/llama-hub/commit/f23c0cc9a21667693d300ceff9e150ca3c2225e7) update github repo reader with timeout - [`3f397ea`](https://github.com/run-llama/llama-hub/commit/3f397ea316de6be965045fcb0aa4697b26b478bd) formatting ### 📊 Changes **3 files changed** (+61 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `llama_hub/github_repo/README.md` (+2 -0) 📝 `llama_hub/github_repo/base.py` (+10 -3) 📝 `llama_hub/github_repo/github_client.py` (+49 -9) </details> ### 📄 Description # Description This PR adds a `timeout` arg to the GithubRepositoryReader to allow users to specify a timeout when making requests. Fixes #846 ## Type of Change Please delete options that are not relevant. - [x] Bug fix / Smaller change # How Has This Been Tested? Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration - [ ] Added new unit/integration tests - [x] I stared at the code and made sure it makes sense # Suggested Checklist: - [x] I have performed a self-review of my own code - [x] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [x] New and existing unit tests pass locally with my changes - [x] I ran `make format; make lint` to appease the lint gods --- <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-15 18:16:45 -05:00
yindo closed this issue 2026-02-15 18:16:45 -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-hub#887