Skip to content

make assemblystream seekable - #14641

Merged
MorrisJobke merged 1 commit into
masterfrom
assemblystream-seek
Mar 15, 2019
Merged

make assemblystream seekable#14641
MorrisJobke merged 1 commit into
masterfrom
assemblystream-seek

Conversation

@icewind1991

Copy link
Copy Markdown
Member

Fixes#14618

@icewind1991icewind1991 added the 3. to review Waiting for reviews label Mar 12, 2019
@icewind1991icewind1991 added this to the Nextcloud 16 milestone Mar 12, 2019
Comment threadapps/dav/lib/Upload/AssemblyStream.php
Signed-off-by: Robin Appelman <robin@icewind.nl>

@rullzerrullzer 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.

Code looks good.
I would welcome more tests

@faily-bot

faily-botBot commented Mar 15, 2019

Copy link
Copy Markdown

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 17055: failure

TESTS=ui-regression

@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.

Code looks good 👍

@MorrisJobke
MorrisJobke merged commit 8ff536f into masterMar 15, 2019
@MorrisJobke
MorrisJobke deleted the assemblystream-seek branch March 15, 2019 11:25
@MorrisJobkeMorrisJobke mentioned this pull request Mar 20, 2019
9 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@icewind1991@rullzer@MorrisJobke