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

[testing] Allow to lock a file without the lock being killed at the end of the request #22823

Merged
merged 2 commits into from
May 30, 2016

Conversation

nickvergessen
Copy link
Contributor

No description provided.

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @jancborchardt and @LukasReschke to be potential reviewers

@nickvergessen nickvergessen force-pushed the add-fake-locking-to-testing-app branch 2 times, most recently from 3c53a14 to 59fd288 Compare March 15, 2016 14:55
@nickvergessen
Copy link
Contributor Author

@PVince81 @icewind1991 please review, so we can finally have smashbox locking tests: owncloud/smashbox#134

@PVince81
Copy link
Contributor

👍

@nickvergessen
Copy link
Contributor Author

@DeepDiver1975 can we have this testing only PR?

@nickvergessen nickvergessen force-pushed the add-fake-locking-to-testing-app branch from 59fd288 to 1479f68 Compare April 11, 2016 07:49
@nickvergessen
Copy link
Contributor Author

Bump, @DeepDiver1975 please review and merge ;)

@PVince81
Copy link
Contributor

@nickvergessen is the "testing" app removed from the tarballs at build time ?
If not we need to make sure it gets done.

@nickvergessen
Copy link
Contributor Author

Yes it is already since 8.2

@PVince81
Copy link
Contributor

@icewind1991 @SergioBertolinSG second review ?

@DeepDiver1975
Copy link
Member

👍

@DeepDiver1975 DeepDiver1975 merged commit 12b482b into master May 30, 2016
@DeepDiver1975 DeepDiver1975 deleted the add-fake-locking-to-testing-app branch May 30, 2016 09:16
@lock
Copy link

lock bot commented Aug 5, 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 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants