Skip to content

Do not log locked files - #16557

Merged
MorrisJobke merged 1 commit into
masterfrom
enh/do_not_log_locked_files
Jul 26, 2019
Merged

Do not log locked files#16557
MorrisJobke merged 1 commit into
masterfrom
enh/do_not_log_locked_files

Conversation

@rullzer

@rullzerrullzer commented Jul 26, 2019

Copy link
Copy Markdown
Member

This is the code doing its job. There is no need to spam the log file
with this.

Signed-off-by: Roeland Jago Douma roeland@famdouma.nl

@rullzer
Do not log locked files
22f9a95
This is the code doing its job. There is no need to spam the log file
with this.
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzerrullzer added this to the Nextcloud 17 milestone Jul 26, 2019
@rullzerrullzer added the 3. to review Waiting for reviews label Jul 26, 2019
MorrisJobke
MorrisJobke approved these changes Jul 26, 2019
View reviewed changes

@MorrisJobkeMorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Makes sense 👍

@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 Jul 26, 2019
@rullzer

rullzer commented Jul 26, 2019

Copy link
Copy Markdown
MemberAuthor

/backport to stable16

@MorrisJobke
MorrisJobke merged commit 3ba33ca into masterJul 26, 2019
@delete-merged-branch
delete-merged-branchBot deleted the enh/do_not_log_locked_files branch July 26, 2019 13:44
@backportbot-nextcloudbackportbot-nextcloudBot mentioned this pull request Jul 26, 2019
Merged
@backportbot-nextcloud

backportbot-nextcloudBot commented Jul 26, 2019

Copy link
Copy Markdown

backport to stable16 in #16564

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nickvergessennickvergessennickvergessen approved these changes
@MorrisJobkeMorrisJobkeMorrisJobke approved these changes
@icewind1991icewind1991Awaiting requested review from icewind1991
@juliusknorrjuliusknorrAwaiting requested review from juliusknorr

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

Nextcloud 17

Development

Successfully merging this pull request may close these issues.

3 participants

@rullzer@nickvergessen@MorrisJobke