Skip to content

Commit 3dea06c

Browse files
authored
Merge pull request #11435 from fvaysh/patch-1
Update review-process.md
2 parents 8093328 + e96e255 commit 3dea06c

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

docs/review-process.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,13 @@ This documentation outlines our current processes for how we prioritize items in
66

77
### General review process
88

9-
We use a first-in, first-out system for reviewing pull requests. The longer a pull request has been open, the higher priority it is for our team to review. In some cases—for example, fixing a high-priority issue or merging low-hanging fruit for a deploy—we will stray from this process and use our best judgement to get higher-impact changes deployed more quickly.
9+
We use a first-in, first-out system for reviewing pull requests. The longer a pull request has been open, the higher the priority it is for our team to review. In some cases—for example, fixing a high-priority issue or merging low-hanging fruit for a deploy—we will stray from this process and use our best judgement to get higher-impact changes deployed more quickly.
1010

1111
We aim to have every new PR reviewed with change requests, merged, or closed within 30 days of opening. As outlined in the following sections, different types of pull requests do have different levels of priority, and this may influence how promptly a pull request is acted on.
1212

1313
### Translations
1414

15-
Translation pull requests are considered high priority. These are when the translated and reviewed files from our translation management platform are submitted as a pull request. They are important to our core value of bringing Ethereum education to as many languages as possible. They also are the accumulation of months of work by translators and reviewers, and any delay to this already long process is undesirable.
15+
Translation pull requests are considered high-priority. These are when the translated and reviewed files from our translation management platform are submitted as a pull request. They are important to our core value of bringing Ethereum education to as many languages as possible. They also are the accumulation of months of work by translators and reviewers, and any delay to this already long process is undesirable.
1616

1717
**Timeline:** PRs should be merged within 4 days of opening (by the next release).
1818

@@ -24,7 +24,7 @@ Bug pull requests can vary from low-priority to high-priority depending on the s
2424

2525
### Features
2626

27-
Any new feature correlated to an approved issue is medium-priority. Similar to translators, code contributors dedicate a large amount of their time to improving [ethereum.org](http://ethereum.org) as a resource and addressing their efforts quickly is desirable. New features that were not discussed, requested or approved do not fall under this timeline.
27+
Any new feature correlated with an approved issue is of medium priority. Similar to translators, code contributors dedicate a large amount of their time to improving [ethereum.org](http://ethereum.org) as a resource and addressing their efforts quickly is desirable. New features that were not discussed, requested or approved do not fall under this timeline.
2828

2929
**Timeline:** Major features should be merged within 30 days of opening. Minor features should be merged within 14 days of opening.
3030

0 commit comments

Comments
 (0)