Skip to content

Fetch GitHub commits concurrently in the triage command - #2567

Merged
JonathanBrouwer merged 1 commit into
rust-lang:mainfrom
Kobzol:triage-parallel-fetch
Sep 3, 2026
Merged

Fetch GitHub commits concurrently in the triage command#2567
JonathanBrouwer merged 1 commit into
rust-lang:mainfrom
Kobzol:triage-parallel-fetch

Conversation

@Kobzol

@Kobzol Kobzol commented Sep 2, 2026

Copy link
Copy Markdown
Member

This should make the command much faster. Provided that it works.

@JonathanBrouwer JonathanBrouwer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm a little worried about github rate-limiting us at some point, but 5 at a time seems like it should be fine?

@JonathanBrouwer
JonathanBrouwer added this pull request to the merge queue Sep 3, 2026
Merged via the queue into rust-lang:main with commit d4b69e1 Sep 3, 2026
15 checks passed
@Kobzol
Kobzol deleted the triage-parallel-fetch branch September 3, 2026 07:17
@Kobzol

Kobzol commented Sep 3, 2026

Copy link
Copy Markdown
Member Author

Based on past experiments, 4-5 usually is fine. But I agree that it can be a problem. If we run into the first issue in production, I'll just revert this.

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