for most contributors, that's all - however, if you are a member of the organization matrix-org, on GitHub, please request a review from matrix.org / Synapse Core.
+
that's it for now, a non-draft pull request will automatically request review from the team;
if you need to update your PR, please avoid rebasing and just add new commits to your branch.
If a developer has requested changes, make these changes and let us know if it is ready for a developer to review again.
+
If a developer has requested changes, make these changes and let us know if it is ready for a developer to review again.
+
+
A pull request is a conversation, if you disagree with the suggestions, please respond and discuss it.
+
+
Create a new commit with the changes.
Please do NOT overwrite the history. New commits make the reviewer's life easier.
@@ -533,6 +537,8 @@ be required.
Back to 1.
+
Once the pull request is ready for review again please re-request review from whichever developer did your initial
+review (or leave a comment in the pull request that you believe all required changes have been done).
Once both the CI and the developers are happy, the patch will be merged into Synapse and released shortly!