Skip to content

Draft: Refactor CI::Config::Entry::Variables

Laura Montemayor requested to merge lm-update-entry-class into master

What does this MR do and why?

Refactors Gitlab::Ci::Config::Entry::Variables into Variable

Screenshots or screen recordings

  • Remove Entry::Variables in lieu of Entry::Variable and use strategies for determining if the variable is simple or complex
  • Make the variables in Entry::Root be a ComposableHash and pass in the Entry::Variable class as a composable_class

With all options for creating a variable, which are:

`variable: 'value'

and

variable: 
   value: 'value'
   description: 'description'

and

variable: 
   value: 'value'

Screenshot_2022-08-16_at_13.19.18.

Validations:

Screenshot_2022-08-16_at_12.34.15.

Screenshot_2022-08-16_at_13.19.48.

Screenshot_2022-08-16_at_13.20.03

Edited by Laura Montemayor

Merge request reports