Skip to content

Rename merge train finder and entity

Allison Brownerequested to merge
298541-4 into master

What does this MR do and why?

Why

As part of migrating to the new domain model which uses the ubiquitous language for merge trains. Related to #298541 (closed)

Today

  • a merge_train has a 1:1 relationship with a merge request. It should be that:
  • a merge_train contains many merge_train_cars
  • merge_train_cars have a 1:1 relationship to merge requests

What

In this MR, we implement the following:

  • Namespace MergeTrainFinder as MergeTrains::TrainFinder
  • Rename Entities::MergeTrain to Entities::MergeTrains::Car

I will change the CODEOWNERS file in a separate MR since it requires approval from development leadership at the highest level and sometimes holds up MRs. Related Associations as well as methods MergeTrain and Car will be renamed in other MRs.

Related to #298541 (closed)

Edited by Allison Browne

Merge request reports