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

handle rate limiting and more repo load messages #2782

Merged
merged 1 commit into from
May 6, 2024

Conversation

GaryPWhite
Copy link
Contributor

@GaryPWhite GaryPWhite commented May 2, 2024

Description

  • more messages when loading repos
  • patched edge cases where sometimes no message would occur
  • rate limiting will now wait to continue loading repositories (before it would fail and continue to the next repo, making large load jobs very difficult)
  • allowed for use of a proxy when checking internet access (http_proxy/HTTP_PROXY is standard in many libraries, including the one in use here)

Future enhancements could include skipping validation on github until checking that a given repo is not in the database -- that's a problem for another day.

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Copy link
Member

@sgoggins sgoggins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@sgoggins sgoggins merged commit 23388e4 into chaoss:main May 6, 2024
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants