Skip to content

Conversation

@marcelklehr
Copy link
Member

Jesus, I've never messed up a pull request so badly. This is the same as #16397 and #16409 but with a signed off commit and it shouldn't fail the tests.

@kesselb kesselb requested review from icewind1991 and rullzer July 17, 2019 13:27
@kesselb kesselb added 3. to review Waiting for reviews bug labels Jul 17, 2019
@kesselb kesselb added this to the Nextcloud 17 milestone Jul 17, 2019
@korelstar
Copy link
Member

And again:
Plesse backport this to NC 15 and 16
;-)

@marcelklehr
Copy link
Member Author

@rullzer Most of the tests are fixed, now

@MorrisJobke MorrisJobke mentioned this pull request Jul 17, 2019
28 tasks
@rullzer
Copy link
Member

rullzer commented Jul 17, 2019

/backport to stable16

@rullzer
Copy link
Member

rullzer commented Jul 17, 2019

/backport to stable15

Copy link
Member

@MorrisJobke MorrisJobke left a comment

Choose a reason for hiding this comment

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

Code makes sense 👍

@MorrisJobke MorrisJobke merged commit 48653d1 into nextcloud:master Jul 17, 2019
@welcome
Copy link

welcome bot commented Jul 17, 2019

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22
Most developers hang out on IRC. So join #nextcloud-dev on Freenode for a chat!

@backportbot-nextcloud
Copy link

backport to stable16 in #16444

@backportbot-nextcloud
Copy link

backport to stable15 in #16445

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

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants