Skip to content

Commit da01773

Browse files
Copilotalexr00
andcommitted
Fix comment - clarify chunk size rationale
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
1 parent 5f35118 commit da01773

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/github/folderRepositoryManager.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -989,9 +989,9 @@ export class FolderRepositoryManager extends Disposable {
989989

990990
// Process branches in chunks to avoid overwhelming the system
991991
// Using a smaller chunk size (10) compared to getLocalPullRequests (100) because:
992-
// - This operation makes GitHub API calls for each branch that doesn't have metadata
993-
// - getLocalPullRequests only reads local config, which is much faster
994-
// - We want to be conservative on first activation to avoid rate limiting
992+
// - This runs on first activation when we don't know how many branches need API calls
993+
// - We want to be conservative to avoid rate limiting
994+
// - The operation is async and non-blocking, so lower throughput is acceptable
995995
const chunkSize = 10;
996996
const associationResults: boolean[] = [];
997997

0 commit comments

Comments
 (0)