Skip to content

Commit 5f35118

Browse files
Copilotalexr00
andcommitted
Address code review feedback - document chunk size rationale
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
1 parent c682ae3 commit 5f35118

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

src/github/folderRepositoryManager.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -988,6 +988,10 @@ export class FolderRepositoryManager extends Disposable {
988988
Logger.debug(`Found ${localBranches.length} local branches to check`, this.id);
989989

990990
// Process branches in chunks to avoid overwhelming the system
991+
// 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
991995
const chunkSize = 10;
992996
const associationResults: boolean[] = [];
993997

0 commit comments

Comments
 (0)