Skip to content

Make it private, not protected, in Share20OCS#29493

Merged
PVince81 merged 1 commit into
masterfrom
make-jp-happy
Nov 7, 2017
Merged

Make it private, not protected, in Share20OCS#29493
PVince81 merged 1 commit into
masterfrom
make-jp-happy

Conversation

@PVince81

@PVince81 PVince81 commented Nov 7, 2017

Copy link
Copy Markdown
Contributor

For #29457 (comment)

@jvillafanez

No backports, master only

@codecov

codecov Bot commented Nov 7, 2017

Copy link
Copy Markdown

Codecov Report

Merging #29493 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #29493   +/-   ##
=========================================
  Coverage     60.85%   60.85%           
  Complexity    17240    17240           
=========================================
  Files          1032     1032           
  Lines         57372    57372           
=========================================
  Hits          34912    34912           
  Misses        22460    22460
Impacted Files Coverage Δ Complexity Δ
apps/files_sharing/lib/API/Share20OCS.php 91.62% <100%> (ø) 149 <3> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fc6dc3d...fa3859d. Read the comment docs.

@PVince81 PVince81 merged commit 8a2cde7 into master Nov 7, 2017
@PVince81 PVince81 deleted the make-jp-happy branch November 7, 2017 19:43
@mrow4a

mrow4a commented Nov 7, 2017

Copy link
Copy Markdown
Contributor

It also makes me happy, not only @jvillafanez 🤣

@PVince81

PVince81 commented Nov 8, 2017

Copy link
Copy Markdown
Contributor Author

@mrow4a I'll think of you next time I make a branch

@lock

lock Bot commented Aug 2, 2019

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock Bot locked as resolved and limited conversation to collaborators Aug 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants