Skip to content

Implement `HasVariable` concern

Shinya Maeda requested to merge feature/sm/34314-add-hasvariable-concern into master

What does this MR do?

This MR adds HasVariable concern for for sharing Ci::Variable core implementation with Group/Project/PipelineSchedule Variables

Are there points in the code the reviewer needs to double check?

Need to check that we don't break the old Ci::Variable functionality.

Why was this MR needed?

This MR is useful for https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/12341 and https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/11078

Screenshots (if relevant)

N/A

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #34314 (closed)

/cc @godfat @ayufan @zj

Edited by Kamil Trzciński

Merge request reports