- if @commits.present?
  .ui-box
    %h5 Commits (#{@commits.count})
    .merge-request-commits
      - if @commits.count > 8
        %ul.first_mr_commits.unstyled
          - @commits.first(8).each do |commit|
            = render "commits/commit", :commit => commit
          %li.bottom
            8 of #{@commits.count} commits displayed.
            %strong
              %a.show_all Click here to show all
        %ul.all_mr_commits.hide.unstyled
          - @commits.each do |commit|
            = render "commits/commit", :commit => commit

      - else
        %ul.unstyled
          - @commits.each do |commit|
            = render "commits/commit", :commit => commit

- else
  %h5
    Nothing to merge from
    %span.label #{@merge_request.source_branch}
    to
    %span.label #{@merge_request.target_branch}
  %br

- unless @commits.empty?
  .ui-box
    %h5 Commits (#{@commits.count})
    .merge-request-commits
      - if @commits.count > 8
        %ul.first_mr_commits.unstyled
          - @commits.first(8).each do |commit|
            = render "commits/commit", :commit => commit
          %li.bottom
            8 of #{@commits.count} commits displayed.
            %strong
              %a.show_all Click here to show all
        %ul.all_mr_commits.hide.unstyled
          - @commits.each do |commit|
            = render "commits/commit", :commit => commit

      - else
        %ul.unstyled
          - @commits.each do |commit|
            = render "commits/commit", :commit => commit

- else
  %h5
    Nothing to merge from
    %span.label #{@merge_request.source_branch}
    to
    %span.label #{@merge_request.target_branch}
  %br

Revert "Invert "unless .empty?" to "if .present?"; unless-else is evil."
Revert "Invert "unless .empty?" to "if .present?"; unless-else is evil."

This reverts commit 7484bcfff7fc8e132d94701b3af47f521ce117c5.
