fix(config): warn if registryUrls is set at top-level in config#41682
fix(config): warn if registryUrls is set at top-level in config#41682viceice merged 5 commits intorenovatebot:mainfrom
registryUrls is set at top-level in config#41682Conversation
viceice
left a comment
There was a problem hiding this comment.
this is not what I had in my mind to fix the issue. it's nearly same as the original PR, where I told that's the wrong solution.
there should be config validation warning instead to not use registry urls at top level config.
later it should be an error because it will almost always cause errors
registryUrls is set at top-level in config
jamietanna
left a comment
There was a problem hiding this comment.
We also need to warn if defaultRegistryUrls
viceice
left a comment
There was a problem hiding this comment.
add same for defaultRegistryUrls
one for global config test another for repo config |
viceice
left a comment
There was a problem hiding this comment.
ok, I don't see a benefit because the check is always done but ok
|
🎉 This PR is included in version 43.56.1 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Changes
Context
Please select one of the following:
registryUrls #41663AI assistance disclosure
Did you use AI tools to create any part of this pull request?
Please select one option and, if yes, briefly describe how AI was used (e.g., code, tests, docs) and which tool(s) you used.
Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via:
The public repository: