RFR: 1437: Fix the OpenJDK official role name and pluralization [v3]

Guoxiong Li gli at openjdk.java.net
Sat May 14 13:15:53 UTC 2022


> Hi all,
> 
> This patch mainly fixes the following message:
> 
> 1. Remove the pluralization if these is 0 reviewer. `no reviews required` --> `no review required`
> 2. Fix the official role name. eg: `reviewer` --> `Reviewer`, `committers` --> `Committers`
> 3. Unify the comment and the progress message. ` (with 1 of role reviewers)` --> `(with at least 1 Reviewer)`
> 4. Fix the pluralization of the comment. ` (with 1 of role reviewers)` --> `(with at least 1 Reviewer)` Note the pluralization.
> 
> Thanks for taking the time to review.
> 
> Best Regards,
> -- Guoxiong

Guoxiong Li has updated the pull request incrementally with one additional commit since the last revision:

  Fix bylaws url.

-------------

Changes:
  - all: https://git.openjdk.java.net/skara/pull/1319/files
  - new: https://git.openjdk.java.net/skara/pull/1319/files/0042b23c..fa250869

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=skara&pr=1319&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=skara&pr=1319&range=01-02

  Stats: 10 lines in 4 files changed: 6 ins; 0 del; 4 mod
  Patch: https://git.openjdk.java.net/skara/pull/1319.diff
  Fetch: git fetch https://git.openjdk.java.net/skara pull/1319/head:pull/1319

PR: https://git.openjdk.java.net/skara/pull/1319


More information about the skara-dev mailing list