-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
feat(self-hosted): convert experimental vars to config options #29137
base: main
Are you sure you want to change the base?
feat(self-hosted): convert experimental vars to config options #29137
Conversation
RENOVATE_X_GITLAB_AUTO_MERGEABLE_CHECK_ATTEMPS, RENOVATE_X_GITLAB_BRANCH_STATUS_DELAY, RENOVATE_X_GITLAB_MERGE_REQUEST_DELAY
Co-authored-by: HonkingGoose <[email protected]>
Co-authored-by: HonkingGoose <[email protected]>
Verified each migration locally, by logging the values of each option at runtime. |
globalOnly: true, | ||
}, | ||
{ | ||
name: 'gitlabAutoMergeableCheckAttempts', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure I understand. Do you mean nesting it inside a platformOptions
object? Does it give any advantage other than making sure all possible ones are located together within config?
Changes
RENOVATE_X_DOCKER_MAX_PAGES
dockerMaxPages
RENOVATE_X_GITLAB_AUTO_MERGEABLE_CHECK_ATTEMPS
gitlabAutoMergeableCheckAttempts
RENOVATE_X_GITLAB_BRANCH_STATUS_DELAY
gitlabBranchStatusDelay
RENOVATE_X_GITLAB_MERGE_REQUEST_DELAY
gitlabMergeRequestDelay
RENOVATE_X_PLATFORM_VERSION
platformVersion
Context
Part of: #27879 (comment)
Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via: