11# Review Guidelines
22
33The following summarises review guidelines that we follow for pull requests in
4- Riot Web and other supporting repos. These are just guidelines (not strict
4+ Element and other supporting repos. These are just guidelines (not strict
55rules) and may be updated over time.
66
77## Code Review
@@ -58,7 +58,7 @@ When reviewing code, here are some things we look for and also things we avoid:
5858
5959## Design and Product Review
6060
61- We want to ensure that all changes to Riot fit with our design and product
61+ We want to ensure that all changes to Element fit with our design and product
6262vision. We often request review from those teams so they can provide their
6363perspective.
6464
@@ -72,11 +72,11 @@ functionality is:
7272
7373As it can be difficult to review design work from looking at just the changed
7474files in a PR, authors should be prepared for Design and / or Product teams to
75- request a link to an ad-hoc build of Riot (hosted anywhere) that can be used for
75+ request a link to an ad-hoc build of Element (hosted anywhere) that can be used for
7676the review. In the future, we [ hope to automate
7777this] ( https://github.com/vector-im/riot-web/issues/12624 ) for every PR.
7878
7979Before starting work on a feature, it's best to ensure your plan aligns well
80- with our vision for Riot . Please chat with the team in
81- [ #riot -dev: matrix .org] ( https://matrix.to/#/#riot -dev:matrix.org ) before you
80+ with our vision for Element . Please chat with the team in
81+ [ #element -dev: matrix .org] ( https://matrix.to/#/#element -dev:matrix.org ) before you
8282start so we can ensure it's something we'd be willing to merge.
0 commit comments