Skip to content

Add `validates :key, uniqueness: { scope: :pipeline_schedule_id }` in `Ci::PipelineScheduleVariable` model

What does this MR do?

This MR adds validates :key, uniqueness: { scope: :pipeline_schedule_id } to Ci::PipelineScheduleVariable model.

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

Is this easy to understand for other devs?

Why was this MR needed?

Originnaly issued at https://gitlab.com/gitlab-org/gitlab-ce/issues/34805. This is one of significant technical debt of PipelineSchedule. Also, this blocks https://gitlab.com/gitlab-org/gitlab-ce/issues/34518

Screenshots (if relevant)

N/A

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Edited by Shinya Maeda

Merge request reports