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

[stable10] Fix files scan repair in bulk warning #29631

Merged
merged 1 commit into from
Nov 20, 2017
Merged

[stable10] Fix files scan repair in bulk warning #29631

merged 1 commit into from
Nov 20, 2017

Conversation

PVince81
Copy link
Contributor

Backport of #29629 to stable10

@PVince81 PVince81 added 3 - To Review p1-urgent Critical issue, need to consider hotfix with just that issue labels Nov 20, 2017
@PVince81 PVince81 added this to the QA milestone Nov 20, 2017
@PVince81 PVince81 self-assigned this Nov 20, 2017
@PVince81 PVince81 mentioned this pull request Nov 20, 2017
Merged
12 tasks
davitol
davitol approved these changes Nov 20, 2017
View reviewed changes
Copy link
Contributor

@davitol davitol left a comment

Choose a reason for hiding this comment

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

Tested and works

@codecov
Copy link

codecov bot commented Nov 20, 2017 •
edited
Loading

Codecov Report

Merging #29631 into stable10 will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@              Coverage Diff               @@
##             stable10   #29631      +/-   ##
==============================================
- Coverage       60.49%   60.49%   -0.01%     
  Complexity      17879    17879              
==============================================
  Files            1064     1064              
  Lines           59521    59522       +1     
==============================================
  Hits            36010    36010              
- Misses          23511    23512       +1
Impacted Files Coverage Δ Complexity Δ
apps/files/lib/Command/Scan.php 0% <0%> (ø) 47 <0> (ø) ⬇️

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 82d92e7...e49ae7d. Read the comment docs.

@PVince81
Copy link
Contributor Author

failed selenium tests unrelated, merging

@PVince81 PVince81 merged commit 346dcd3 into stable10 Nov 20, 2017
@PVince81 PVince81 deleted the stable10-scan-repair-all-message branch November 20, 2017 16:16
@lock
Copy link

lock bot commented Aug 1, 2019

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 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tomneedham tomneedham tomneedham approved these changes

@davitol davitol davitol approved these changes

Assignees

@PVince81 PVince81

Labels
3 - To Review p1-urgent Critical issue, need to consider hotfix with just that issue status/STALE
Projects
None yet
Milestone
10.0.4
Development

Successfully merging this pull request may close these issues.
4 participants
@PVince81 @tomneedham @davitol @ownclouders