Commit ebef1a84 authored by Grzegorz Bizon's avatar Grzegorz Bizon

Remove change to MWBS in code review guideline

[ci skip]
parent 85c4aa4a
...@@ -70,7 +70,7 @@ experience, refactors the existing code). Then: ...@@ -70,7 +70,7 @@ experience, refactors the existing code). Then:
- After a round of line notes, it can be helpful to post a summary note such as - After a round of line notes, it can be helpful to post a summary note such as
"LGTM :thumbsup:", or "Just a couple things to address." "LGTM :thumbsup:", or "Just a couple things to address."
- Avoid accepting a merge request before the build succeeds. Of course, "Merge - Avoid accepting a merge request before the build succeeds. Of course, "Merge
When Pipeline Succeeds" is fine. When Build Succeeds" (MWBS) is fine.
- If you set the MR to "Merge When Build Succeeds", you should take over - If you set the MR to "Merge When Build Succeeds", you should take over
subsequent revisions for anything that would be spotted after that. subsequent revisions for anything that would be spotted after that.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment