Reuse variables builder for Ci::Config rules evaluation
What does this MR do and why?
Related to #352587 (closed)
The initial plan was to couple only the secret variables under a single method in the variables builder, but I think this makes more sense.
The plan | ||
---|---|---|
Refactor instance secret variables | !79435 (merged) | |
Refactor project secret variables | !79850 (merged) | |
Refactor group secret variables | !79936 (merged) | |
Bundle them under a single method | !79935 (merged) | this one |
Screenshots or screen recordings
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Edited by Marius Bobin