Evaluate pr/branch limits per-branch

[rarkins]

Describe the proposed change(s).

Current, limit are calculated prior to branch execution:

renovate/lib/workers/repository/process/write.ts

Lines 130 to 138 in ed7b17a

	const prsRemaining = await getPrsRemaining(config, branches);
	logger.debug(`Calculated maximum PRs remaining this run: ${prsRemaining}`);
	setMaxLimit('PullRequests', prsRemaining);
	const branchesRemaining = await getBranchesRemaining(config, branches);
	logger.debug(
	`Calculated maximum branches remaining this run: ${branchesRemaining}`,
	);
	setMaxLimit('Branches', branchesRemaining);

And evaluated within branch execution such as

renovate/lib/workers/repository/update/branch/index.ts

Lines 212 to 224 in ed7b17a

	if (
	!branchExists &&
	isLimitReached('Branches') &&
	!dependencyDashboardCheck &&
	!config.isVulnerabilityAlert
	) {
	logger.debug('Reached branch limit - skipping branch creation');
	return {
	branchExists,
	prNo: branchPr?.number,
	result: 'branch-limit-reached',
	};
	}

Much of the logic is also inside https://github.com/renovatebot/renovate/blob/main/lib/workers/repository/process/limits.ts

Instead of having global limits as today, we should instead allow separate limits per-branch, and evaluate those limits per-branch instead. This means removing these global limit settings:

• renovate/lib/workers/repository/process/write.ts

  Line 132 in ed7b17a

  setMaxLimit('PullRequests', prsRemaining);

• renovate/lib/workers/repository/process/write.ts

  Line 138 in ed7b17a

  setMaxLimit('Branches', branchesRemaining);

And instead passing the relevant limit like prBranchLimit and prConcurrentLimit through to the "is limit reached?" functions.

Ideally the point in our workers where these limits are evaluated is kept the same as today so that there is "less changes" and risk of breaking things. And so the change is that the limits are passed through rather than used globally.

Note: commits will still be global

[rarkins]

I've begun initial refactorings for this in https://github.com/renovatebot/renovate/compare/feat/per-branch-limits?expand=1

[RahulGautamSingh]

Duplicate: #10546

[rarkins]

Different actually. I wanted the first work to be evaluating per branch. Second work is allowing multiple groups per branch

[renovate-release]

🎉 This issue has been resolved in version 39.119.0 🎉

The release is available on:

• GitHub release
• npm package (@latest dist-tag)
• 39.119.0

Your semantic-release bot 📦🚀