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.

Sign up for GitHub

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

Jump to bottom

[stable13] cleanup locks in scanner on error #9566

Merged
merged 1 commit into from
May 23, 2018
Merged

[stable13] cleanup locks in scanner on error #9566

merged 1 commit into from
May 23, 2018

Conversation

rullzer
Copy link
Member

@rullzer rullzer commented May 23, 2018

Backport of #9395

+1 from me

@icewind1991 @rullzer
cleanup locks in scanner on error
874e312 
Signed-off-by: Robin Appelman <robin@icewind.nl>
@rullzer rullzer added enhancement 3. to review Waiting for reviews labels May 23, 2018
@rullzer rullzer added this to the Nextcloud 13.0.3 milestone May 23, 2018
@rullzer rullzer requested a review from MorrisJobke May 23, 2018 18:09
@MorrisJobke MorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels May 23, 2018
@codecov
Copy link

codecov bot commented May 23, 2018 •
edited
Loading

Codecov Report

Merging #9566 into stable13 will increase coverage by <.01%.
The diff coverage is 90%.

@@              Coverage Diff              @@
##             stable13   #9566      +/-   ##
=============================================
+ Coverage        51.4%   51.4%   +<.01%     
  Complexity      25089   25089              
=============================================
  Files            1612    1612              
  Lines           95484   95490       +6     
  Branches         1376    1376              
=============================================
+ Hits            49081   49085       +4     
- Misses          46403   46405       +2
Impacted Files Coverage Δ Complexity Δ
lib/private/Files/Cache/Scanner.php 82.27% <90%> (-0.28%) 115 <0> (ø)
lib/private/Files/Cache/Propagator.php 94.93% <0%> (-1.27%) 16% <0%> (ø)
apps/dav/lib/Connector/Sabre/Directory.php 66.45% <0%> (+0.85%) 65% <0%> (ø) ⬇️

@MorrisJobke MorrisJobke merged commit fe20bb4 into stable13 May 23, 2018
@MorrisJobke MorrisJobke deleted the backport/9395/stable13 branch May 23, 2018 20:57
This was referenced May 31, 2018
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MorrisJobke MorrisJobke MorrisJobke approved these changes

Assignees
No one assigned
Labels
4. to release Ready to be released and/or waiting for tests to finish enhancement
Projects
None yet
Milestone
Nextcloud 13.0.3
Development

Successfully merging this pull request may close these issues.
3 participants
@rullzer @MorrisJobke @icewind1991