Skip to content

[stable12] cleanup locks in scanner on error - #9567

Merged
rullzer merged 1 commit into
stable12from
backport/9395/stable12
May 23, 2018
Merged

[stable12] cleanup locks in scanner on error#9567
rullzer merged 1 commit into
stable12from
backport/9395/stable12

Conversation

@rullzer

Copy link
Copy Markdown
Member

Backport of #9395

+1 from me

Signed-off-by: Robin Appelman <robin@icewind.nl>
@rullzerrullzer added enhancement 3. to review Waiting for reviews labels May 23, 2018
@rullzerrullzer added this to the Nextcloud 12.0.8 milestone May 23, 2018
@rullzer
rullzer requested a review from MorrisJobkeMay 23, 2018 18:10
@MorrisJobkeMorrisJobke 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

codecovBot commented May 23, 2018

Copy link
Copy Markdown

Codecov Report

Merging #9567 into stable12 will increase coverage by <.01%.
The diff coverage is 90%.

@@ Coverage Diff @@## stable12 #9567 +/- ##
==============================================
+ Coverage 53.93% 53.93% +<.01% 
Complexity 22813 22813 ==============================================
Files 1389 1389 Lines 87398 87401 +3 Branches 1331 1331 ==============================================
+ Hits 47134 47141 +7 + Misses 40264 40260 -4
Impacted FilesCoverage ΔComplexity Δ
lib/private/Files/Cache/Scanner.php81.33% <90%> (-0.29%)109 <0> (ø)
version.php0% <0%> (ø)0% <0%> (ø)⬇️
core/js/js.js61.83% <0%> (+0.55%)0% <0%> (ø)⬇️
lib/private/Files/Cache/Propagator.php96.2% <0%> (+1.26%)16% <0%> (ø)⬇️

@rullzer
rullzer merged commit 3ebed2b into stable12May 23, 2018
@rullzer
rullzer deleted the backport/9395/stable12 branch May 23, 2018 21:04
@MorrisJobkeMorrisJobke mentioned this pull request May 31, 2018
2 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishenhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@rullzer@MorrisJobke@icewind1991