From 2de90dabff56c56902d3bec2c6f7de3781242126 Mon Sep 17 00:00:00 2001 From: Adam Niedzielski Date: Tue, 28 Feb 2017 11:12:48 +0000 Subject: Clarify when to create EE compatibility MR in code review process --- doc/development/limit_ee_conflicts.md | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'doc/development') diff --git a/doc/development/limit_ee_conflicts.md b/doc/development/limit_ee_conflicts.md index 2d82b09f301..e3568b65b18 100644 --- a/doc/development/limit_ee_conflicts.md +++ b/doc/development/limit_ee_conflicts.md @@ -50,6 +50,12 @@ Notes: asking a GitLab developer to do it once the merge request is merged. - If you branch is more than 500 commits behind `master`, the job will fail and you should rebase your branch upon latest `master`. +- Code reviews for merge requests often consist of multiple iterations of + feedback and fixes. There is no need to update your EE MR after each + iteration. Instead, create an EE MR as soon as you see the + `rake ee_compat_check` job failing and update it after the CE MR is merged. + This helps to identify significant conflicts sooner, but also reduces the + number of times you have to resolve conflicts. ## Possible type of conflicts -- cgit v1.2.3