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: dynamic manager config #5392
Conversation
I think we may still have a circular import problem however it's hidden by the dynamic manager
|
can we move it to new dynamic import syntax? then we need to call a promise await on startup to init managers instead of auto init. |
ok, platform should be loaded dynamically too |
renovate/lib/platform/index.ts Lines 12 to 14 in 987e0a2
|
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.
LGTM, hopefully no mistake while copy'n'paste config. 😅
I think the JSON schema works as a good check in this case. From what I can see, the only field that changed was the "description" one. |
🎉 This PR is included in version 19.126.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Move manager definitions and default configs from lib/config/definitions to lib/manager/x
Closes #5279