Searched refs:reviewer (Results 1 – 25 of 33) sorted by relevance
12
| /third_party/typescript/scripts/ |
| D | request-pr-review.mjs | 6 string: ["token", "pull", "reviewer", "owner", "repo"],
15 reviewer: process.env.REQUESTED_REVIEWER,
25 if (!options.token || !options.pull || !options.reviewer || !options.owner || !options.repo) {
36 const reviewers = Array.isArray(options.reviewer) ? options.reviewer : [options.reviewer];
68 --reviewer <reviewer> The GH username of reviewer to add. May be specified multiple times.
|
| /third_party/node/deps/v8/tools/release/ |
| D | auto_push.py | 78 "--reviewer", self._options.reviewer,
98 if not options.author or not options.reviewer: # pragma: no cover
|
| D | common_includes.py | 663 reviewer = None
664 if self._options.reviewer:
665 print("Using account %s for review." % self._options.reviewer)
666 reviewer = self._options.reviewer
673 if not reviewer and not tbr_reviewer:
678 reviewer = self.ReadLine()
680 self.GitUpload(reviewer, self._options.force_upload,
|
| D | git_recipes.py | 208 def GitUpload(self, reviewer="", force=False, cq=False, argument
213 if reviewer:
214 args += ["-r", Quoted(reviewer)]
|
| D | auto_roll.py | 155 message.append("R=%s" % self._options.reviewer)
208 if not options.author or not options.reviewer:
|
| D | create_release.py | 206 if not options.author or not options.reviewer:
|
| /third_party/skia/third_party/externals/angle2/doc/ |
| D | CodeReviewProcess.md | 10 changes even if you are not a designated reviewer!
23 reviewer, you can request a follow-up CL to address the style issue if
36 * ANGLE has a 2-reviewer policy for CLs. This means all changes should get
39 reviewer to clear the CL.
|
| D | TestingAndProcesses.md | 62 CLs that only change dEQP or angle_end2end_tests expectations don't require a human reviewer and ca…
65 …-stamper@appspot.gserviceaccount.com as the reviewer on the expectations CL. Please also CC a huma…
|
| D | ContributingCode.md | 59 ask a member of the group to submit the tryjob for you. Add jmadill or geofflang as a reviewer
177 * If you are not a committer, ask your reviewer to submit the change list.
238 and the fixed/final Patchset. Notice how a reviewer can easily see the fix to the original CL.
|
| /third_party/skia/site/docs/dev/contrib/ |
| D | submit.md | 71 ### Find a reviewer
73 Ideally, the reviewer is someone who is familiar with the area of code you are
125 review and click **Reply**. Enter at least one reviewer's email address. Now add
130 in the upper right. _Hint_: You can add -r reviewer@example.com --send-mail to
138 Code reviews are an important part of the engineering process. The reviewer will
144 You will likely get email back from the reviewer with comments. Fix these and
159 notification (it is helpful to tell the reviewer what you did with respect to
160 each of their comments). When the reviewer is happy with your patch, they will
|
| D | cqkeywords.md | 29 The CQ will run through its list of verifiers (reviewer check, trybots, tree check,
|
| /third_party/protobuf/ |
| D | CONTRIBUTING.md | 71 * Unless your PR is trivial, you should expect there will be reviewer comments
84 The reviewer will help you if there are test failures that seem not related
97 request affects, allowing us to better identify appropriate reviewer, create
|
| /third_party/node/deps/v8/ |
| D | OWNERS | 1 # Eng reviewer. Please reach out before adding new top-level directories.
|
| /third_party/skia/site/docs/user/ |
| D | issue-tracker.md | 29 on [status.skia.org](https://status.skia.org) and as a reviewer on the roll CL.
|
| /third_party/node/ |
| D | glossary.md | 9 * RSLGTM: "Rubber-stamp looks good to me". The reviewer approving without doing
|
| /third_party/skia/third_party/externals/swiftshader/third_party/SPIRV-Tools/ |
| D | CONTRIBUTING.md | 67 The reviewer can either approve your PR or request changes. If changes are
71 Adding new commits makes it easier for the reviewer to see what has changed
77 After the PR has been reviewed it is the job of the reviewer to merge the PR.
|
| /third_party/spirv-tools/ |
| D | CONTRIBUTING.md | 67 The reviewer can either approve your PR or request changes. If changes are
71 Adding new commits makes it easier for the reviewer to see what has changed
77 After the PR has been reviewed it is the job of the reviewer to merge the PR.
|
| /third_party/skia/third_party/externals/spirv-tools/ |
| D | CONTRIBUTING.md | 67 The reviewer can either approve your PR or request changes. If changes are
71 Adding new commits makes it easier for the reviewer to see what has changed
77 After the PR has been reviewed it is the job of the reviewer to merge the PR.
|
| /third_party/skia/site/docs/dev/chrome/ |
| D | _index.md | 25 a reviewer on the roll CL.
|
| /third_party/googletest/googletest/scripts/ |
| D | upload.py | 1321 for reviewer in options.reviewers.split(','):
1322 if "@" in reviewer and not reviewer.split("@")[1].count(".") == 1:
1323 ErrorExit("Invalid email address: %s" % reviewer)
|
| /third_party/jerryscript/docs/ |
| D | 08.CODING-STANDARDS.md | 98 * Although this is correct, a reviewer might request
665 * declarations. A reviewer might request such a change. */
|
| /third_party/rust/crates/nix/ |
| D | CONTRIBUTING.md | 98 pull request has been reviewed, the reviewer will leave a comment like
|
| /third_party/curl/docs/ |
| D | CODE_REVIEW.md | 36 deliberately and carefully. If not, a reviewer must help the author to realize
|
| /third_party/skia/third_party/externals/abseil-cpp/ |
| D | CONTRIBUTING.md | 101 * Unless your PR is trivial, you should expect there will be reviewer comments
|
| /third_party/skia/modules/canvaskit/ |
| D | README.md | 154 12. Send out CL for review. Feel free to point the reviewer at these steps.
|
12