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
refactor(repo/init): return additional raw config from detectRepoFileConfig #17021
refactor(repo/init): return additional raw config from detectRepoFileConfig #17021
Conversation
…poFileConfig - revert caching
reverted |
9c7623a
to
c9a470e
Compare
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
learned something new, i pushed my changes but forgot to commit suggestions and suggestions got outdated which means they can't be commited. |
# Conflicts: # lib/workers/repository/init/merge.ts
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
This reverts commit 4275c39.
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
8f2a43d
to
073406f
Compare
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.
needs deconflicting
# Conflicts: # lib/workers/repository/init/merge.spec.ts
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.
otherwise LGTM
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
you can find the blocking open discussion yourself 😛 |
🎉 This PR is included in version 32.165.2 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Changes
detectRepoFileConfig
will now also return the raw config filerawConfigFile
to theRepoFileConfig
type (return value ofdetectRepoFileConfig
)cache (memCache
) return values fordetectRepoFileConfig
As each repo has only one valid renovate config,
it might be beneficial to cache it.
detectRepoFileConfig
is also used in the config migration flow and doing so will save some api/fs calls.once this is accepted, it can be utilized in order to resolve the following issue
The main problem with #16664 is the fact that we are trying to load a local copy of the config when the repo wasnt cloned.
detectRepoFileConfig
does it already, so it can be cached and can be used instead.Context
Documentation (please check one with an [x])
How I've tested my work (please tick one)
I have verified these changes via: