Skip to content

Test hs/upload limits - #466

Merged
richvdh merged 6 commits into
developfrom
hs/upload-limits
Aug 16, 2018
Merged

richvdh merged 6 commits into
developfrom
hs/upload-limits

Conversation

@Half-Shot

@Half-Shot Half-Shot commented Jul 18, 2018 •

Copy link
Copy Markdown
Contributor

Changed branch name to apparently make sytest happy which means this follows on from #456.

@Half-Shot

Copy link
Copy Markdown
Contributor Author

Well, I must be doing something wrong because this is now the correct branch name, the full_uri is correct and I'm fairly certain that path comes back with a config.

@richvdh
richvdh requested a review from a team July 26, 2018 14:45
@richvdh

richvdh commented Jul 27, 2018

Copy link
Copy Markdown
Member

@matrixbot retest this please

@richvdh

richvdh commented Jul 27, 2018

Copy link
Copy Markdown
Member

[this is supposed to be testing matrix-org/synapse#3184]

@richvdh

richvdh commented Jul 27, 2018

Copy link
Copy Markdown
Member

Well, I must be doing something wrong because this is now the correct branch name, the full_uri is correct and I'm fairly certain that path comes back with a config.

the reason it didn't pick up your synapse branch is that the synapse branch is on a fork of the repo rather than the matrix.org one.

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

lgtm, pending merge of matrix-org/synapse#3184

@richvdh

richvdh commented Aug 16, 2018

Copy link
Copy Markdown
Member

@matrixbot retest this please

@richvdh
richvdh merged commit 6ac4435 into develop Aug 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants