Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use ::class and fix missing doc #6992

Merged
merged 1 commit into from
Oct 30, 2017
Merged

Conversation

nickvergessen
Copy link
Member

Signed-off-by: Joas Schilling coding@schilljs.com

Signed-off-by: Joas Schilling <coding@schilljs.com>
@codecov
Copy link

codecov bot commented Oct 28, 2017

Codecov Report

Merging #6992 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #6992      +/-   ##
============================================
- Coverage     52.83%   52.83%   -0.01%     
  Complexity    22802    22802              
============================================
  Files          1442     1442              
  Lines         88563    88563              
  Branches       1349     1349              
============================================
- Hits          46790    46788       -2     
- Misses        41773    41775       +2
Impacted Files Coverage Δ Complexity Δ
lib/private/Files/Storage/Local.php 78.19% <100%> (-0.48%) 86 <0> (ø)
lib/private/Files/Cache/Propagator.php 94.93% <0%> (-1.27%) 16% <0%> (ø)

@MorrisJobke MorrisJobke merged commit d18a60d into master Oct 30, 2017
@MorrisJobke MorrisJobke deleted the 2-lines-for-better-code branch October 30, 2017 15:18
@MorrisJobke MorrisJobke added this to the Nextcloud 13 milestone Oct 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants