Skip to content
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

Bump Renovate's prConcurrentLimit to 8 #825

Merged
merged 2 commits into from
May 13, 2022
Merged

Conversation

NimJay
Copy link
Collaborator

@NimJay NimJay commented May 13, 2022

Background

  • We currently have 4 Renovate PRs open that need further investigation (b/c of errors or incomplete errors).
  • In the meantime, let's let in more parallel PRs from Renovate Bot.

Change Summary

  • Allow Renovate Bot to create 8 PRs concurrently.

Testing Procedure

  • We will merge this PR, and wait for Renovate Bot to create a max of 8 concurrent PRs.

@NimJay NimJay requested a review from a team as a code owner May 13, 2022 15:17
@NimJay NimJay changed the title Bump Renovate's prConcurrentLimit to 10 Bump Renovate's prConcurrentLimit to 8 May 13, 2022
@@ -12,7 +12,7 @@
"constraints": {
"python": "~=3.7.0"
},
"prConcurrentLimit":5,
"prConcurrentLimit":8,
Copy link
Member

@bourgeoisor bourgeoisor May 13, 2022

Choose a reason for hiding this comment

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

Description and commit message says 10, code says 8, which is it supposed to be?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good catch! Will fix. :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I just corrected the commit message to "8" (sorry, had to do a git commit-amend and a git force-push).

@github-actions
Copy link

🚲 PR staged at http://34.67.13.231

@NimJay NimJay force-pushed the nimjay-bump-renovate-limit branch from 856c53b to ab9b6c7 Compare May 13, 2022 15:22
@github-actions
Copy link

🚲 PR staged at http://34.67.13.231

@github-actions
Copy link

🚲 PR staged at http://34.67.13.231

Copy link
Member

@bourgeoisor bourgeoisor left a comment

Choose a reason for hiding this comment

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

LGTM

@NimJay NimJay merged commit 03ca0f9 into main May 13, 2022
@bourgeoisor bourgeoisor deleted the nimjay-bump-renovate-limit branch April 21, 2023 01:35
D-Mwanth pushed a commit to D-Mwanth/microservices-demo that referenced this pull request Mar 6, 2024
Co-authored-by: Olivier Bourgeois <3271352+bourgeoisor@users.noreply.github.com>
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.

None yet

3 participants