-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
GitHub OAuth: skip conda-forge
organization when syncing
#8979
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
Comments
That makes sense to me 👍. I'll see if we have that data on the first request and make the decision there before start consuming the resulting pages |
I took a quick look at this today and I don't think it's possible to implement it without a considerable refactor.
Even making those changes, I'm not 100% sure that this will work 😄 . Ideally, we would like to improve step 1) as much as possible but I didn't find a way to do that yet. |
Currently, we are using the default pagination that GitHub offers, which is 30. This commit increases the page size to 100 when fetching organization for a user and repositories for each organizations. Note we are already using 100 page size when fetching repositories for a user. Related to #8979
I opened #9020 to increase the page size to 100 when dealing with organizations and their repositories. This is not the best fix, but at least it will reduce the number of requests required. |
Currently, we are using the default pagination that GitHub offers, which is 30. This commit increases the page size to 100 when fetching organization for a user and repositories for each organizations. Note we are already using 100 page size when fetching repositories for a user. Related to #8979
conda-forge
on GitHub has a lot of repositories and that produces different issues on our side. The most recent one was that our Celery task was finishing due to timeout and the sync wasn't performed.Since nobody (?) imports projects from
conda-forge
into Read the Docs, we thought it could be a good idea to skip it from the synchronization. This will reduce the time required considerably and also the database's size.Related #8974
The text was updated successfully, but these errors were encountered: