Skip to content

Code style changes from downstream - #1821

Merged
MorrisJobke merged 1 commit into
masterfrom
downstream-26366
Oct 20, 2016
Merged

Code style changes from downstream#1821
MorrisJobke merged 1 commit into
masterfrom
downstream-26366

Conversation

@MorrisJobke

Copy link
Copy Markdown
Member

cc @rullzer@nickvergessen@icewind1991

@MorrisJobkeMorrisJobke added this to the Nextcloud 11.0 milestone Oct 20, 2016
@mention-bot

Copy link
Copy Markdown

@MorrisJobke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DeepDiver1975, @PVince81 and @nickvergessen to be potential reviewers.

@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

Our fix for this issue is in:

11: #1740
10: #1798
9: #1799

@nickvergessen

Copy link
Copy Markdown
Member

Fine by me, but as said. No backport

@nickvergessennickvergessen changed the title Sanitize length headers when validating quotaCode style changes from downstreamOct 20, 2016
@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

Fine by me, but as said. No backport

Is this a +1 then? 😉

@LukasReschke

Copy link
Copy Markdown
Member

LGTM

@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

I tested this and it works 👍

@MorrisJobke
MorrisJobke merged commit 98c8464 into masterOct 20, 2016
@MorrisJobke
MorrisJobke deleted the downstream-26366 branch October 20, 2016 18:18
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.

5 participants

@MorrisJobke@mention-bot@nickvergessen@LukasReschke@DeepDiver1975