From 018e33ef3dafdd49cde4868353c8fcdcbd456b59 Mon Sep 17 00:00:00 2001 From: Philip Chimento Date: Thu, 25 Aug 2022 17:08:41 -0700 Subject: [PATCH] Fix pre-existing linter errors --- stage-3-review.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/stage-3-review.md b/stage-3-review.md index db5303c..01ae83a 100644 --- a/stage-3-review.md +++ b/stage-3-review.md @@ -7,10 +7,11 @@ According to [the TC39 process document](https://tc39.es/process-document), to a Reviewers for Stage 3 volunteer as such in TC39 meetings when discussing a proposal. If the proposal is ready, the champion can ask the committee for reviewers at the same meeting as it advances to Stage 2. Sizable proposals are encouraged to have multiple reviewers. Different reviewers can provide different perspectives. Reviewer feedback is tracked in a GitHub issue in the proposal repository ([example](https://github.com/tc39/proposal-regexp-unicode-property-escapes/issues/3)). It may also be useful to send out a reminder email ahead of the meeting to the reviewers. - + ## Performing a review When reviewing a TC39 proposal, read the specification text, README and other documentation carefully. Some questions which may be useful to consider in a review: + - Does the proposal address its stated motivation? - Is this language feature intuitive and learnable? - Does the proposal fit in well when combined with other JavaScript language features (both current and proposed, a.k.a. "cross-cutting concerns")? Will the interactions between features be surprising or strange? @@ -22,4 +23,3 @@ When reviewing a TC39 proposal, read the specification text, README and other do To give your feedback to the champion group, file issues in GitHub, or make pull requests with proposed changes. Reviews are expected to be performed at least two weeks in advance of the TC39 meeting. -