# HG changeset patch # User Marcin Kuzminski # Date 2019-11-19 15:38:56 # Node ID 697a75c317a455358269b4a7c87f5c591dbc3540 # Parent 95ca7a00ec242061dc7a76e0a895dc2284e460db pull-requests: added nicer formatting for merge conflicting files diff --git a/rhodecode/apps/repository/tests/test_repo_pullrequests.py b/rhodecode/apps/repository/tests/test_repo_pullrequests.py --- a/rhodecode/apps/repository/tests/test_repo_pullrequests.py +++ b/rhodecode/apps/repository/tests/test_repo_pullrequests.py @@ -261,7 +261,7 @@ class TestPullrequestsView(object): True, True, '', MergeFailureReason.MISSING_TARGET_REF, metadata={'target_ref': PullRequest.unicode_to_reference(unicode_reference)}) response.assert_response().element_contains( - 'span[data-role="merge-message"]', merge_resp.merge_status_message) + 'div[data-role="merge-message"]', merge_resp.merge_status_message) def test_comment_and_close_pull_request_custom_message_approved( self, pr_util, csrf_token, xhr_header): diff --git a/rhodecode/lib/vcs/backends/git/repository.py b/rhodecode/lib/vcs/backends/git/repository.py --- a/rhodecode/lib/vcs/backends/git/repository.py +++ b/rhodecode/lib/vcs/backends/git/repository.py @@ -990,7 +990,7 @@ class GitRepository(BaseRepository): except RepositoryError as e: log.exception('Failure when doing local merge on git shadow repo') if isinstance(e, UnresolvedFilesInRepo): - metadata['unresolved_files'] = 'file: ' + (', file: '.join(e.args[0])) + metadata['unresolved_files'] = '\n* conflict: ' + ('\n * conflict: '.join(e.args[0])) merge_possible = False merge_failure_reason = MergeFailureReason.MERGE_FAILED diff --git a/rhodecode/lib/vcs/backends/hg/repository.py b/rhodecode/lib/vcs/backends/hg/repository.py --- a/rhodecode/lib/vcs/backends/hg/repository.py +++ b/rhodecode/lib/vcs/backends/hg/repository.py @@ -830,7 +830,7 @@ class MercurialRepository(BaseRepository except RepositoryError as e: log.exception('Failure when doing local merge on hg shadow repo') if isinstance(e, UnresolvedFilesInRepo): - metadata['unresolved_files'] = 'file: ' + (', file: '.join(e.args[0])) + metadata['unresolved_files'] = '\n* conflict: ' + ('\n * conflict: '.join(e.args[0])) merge_possible = False merge_failure_reason = MergeFailureReason.MERGE_FAILED diff --git a/rhodecode/templates/pullrequests/pullrequest_merge_checks.mako b/rhodecode/templates/pullrequests/pullrequest_merge_checks.mako --- a/rhodecode/templates/pullrequests/pullrequest_merge_checks.mako +++ b/rhodecode/templates/pullrequests/pullrequest_merge_checks.mako @@ -18,14 +18,14 @@ % for pr_check_key, pr_check_details in c.pr_merge_errors.items(): <% pr_check_type = pr_check_details['error_type'] %>
  • - - - ${pr_check_details['message']} +
    + - ${pr_check_details['message']} % if pr_check_key == 'todo': % for co in pr_check_details['details']: ${'' if loop.last else ','} % endfor % endif - +
  • % endfor