Commit 58b57f0c authored by Phil Hughes's avatar Phil Hughes

Merge branch 'patch-53' into 'master'

Don't suggest creating merge commit when using FF merge

See merge request gitlab-org/gitlab!22834
parents 4d52d55f 4062ba66
...@@ -127,7 +127,7 @@ export default { ...@@ -127,7 +127,7 @@ export default {
</button> </button>
<span v-if="!rebasingError" class="bold">{{ <span v-if="!rebasingError" class="bold">{{
__( __(
'Fast-forward merge is not possible. Rebase the source branch onto the target branch or merge target branch into source branch to allow this merge request to be merged.', 'Fast-forward merge is not possible. Rebase the source branch onto the target branch.',
) )
}}</span> }}</span>
<span v-else class="bold danger">{{ rebasingError }}</span> <span v-else class="bold danger">{{ rebasingError }}</span>
......
---
title: Improve fast-forward merge is not possible message
merge_request: 22834
author: Ben Bodenmiller
type: other
...@@ -9291,7 +9291,7 @@ msgstr "" ...@@ -9291,7 +9291,7 @@ msgstr ""
msgid "Fast-forward merge is not possible. Rebase the source branch onto %{targetBranch} to allow this merge request to be merged." msgid "Fast-forward merge is not possible. Rebase the source branch onto %{targetBranch} to allow this merge request to be merged."
msgstr "" msgstr ""
msgid "Fast-forward merge is not possible. Rebase the source branch onto the target branch or merge target branch into source branch to allow this merge request to be merged." msgid "Fast-forward merge is not possible. Rebase the source branch onto the target branch."
msgstr "" msgstr ""
msgid "Fast-forward merge without a merge commit" msgid "Fast-forward merge without a merge commit"
......
...@@ -45,10 +45,8 @@ describe('Merge request widget rebase component', () => { ...@@ -45,10 +45,8 @@ describe('Merge request widget rebase component', () => {
expect(text).toContain('Fast-forward merge is not possible.'); expect(text).toContain('Fast-forward merge is not possible.');
expect(text.replace(/\s\s+/g, ' ')).toContain( expect(text.replace(/\s\s+/g, ' ')).toContain(
'Rebase the source branch onto the target branch or merge target', 'Rebase the source branch onto the target branch.',
); );
expect(text).toContain('branch into source branch to allow this merge request to be merged.');
}); });
it('it should render error message when it fails', done => { it('it should render error message when it fails', done => {
......
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