Skip to content

Conversation

@danxuliu
Copy link
Member

Although #7256 was merged cleanly some of the changes really conflicted with those introduced by the last commit of #7251, and this broke the appearance of the author row of comments. This pull request fixes those silent conflicts and restores the appearance of the author row.

Although #7256 was merged cleanly some of the changes really conflicted
with those introduced by the last commit of #7251, and this broke the
appearance of the author row of comments. This commit fixes those silent
conflicts and restores the appearance of the author row.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@danxuliu danxuliu added this to the Nextcloud 13 milestone Nov 28, 2017
@codecov
Copy link

codecov bot commented Nov 28, 2017

Codecov Report

Merging #7316 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #7316   +/-   ##
=========================================
  Coverage     50.86%   50.86%           
  Complexity    24538    24538           
=========================================
  Files          1584     1584           
  Lines         93794    93794           
  Branches       1358     1358           
=========================================
  Hits          47705    47705           
  Misses        46089    46089
Impacted Files Coverage Δ Complexity Δ
lib/private/Server.php 82.62% <0%> (-0.12%) 126% <0%> (ø)
lib/private/Security/CertificateManager.php 92.07% <0%> (+0.99%) 39% <0%> (ø) ⬇️

@rullzer rullzer requested a review from skjnldsv November 29, 2017 13:20
Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Damn, nice catch!

@rullzer rullzer merged commit f03aa04 into master Nov 29, 2017
@rullzer rullzer deleted the fix-silent-conflicts-due-to-merging-7256-after-7251 branch November 29, 2017 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants