Skip to content

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.

Edited by Marius Bobin

Merge request reports

Loading