You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dolphinscheduler.apache.org by zh...@apache.org on 2022/01/20 10:13:00 UTC
[dolphinscheduler-website] branch master updated: [community] Add how to review issue or PR (#648)
This is an automated email from the ASF dual-hosted git repository.
zhongjiajie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/dolphinscheduler-website.git
The following commit(s) were added to refs/heads/master by this push:
new 1004927 [community] Add how to review issue or PR (#648)
1004927 is described below
commit 1004927d44bb138448771a0e3e0bdf0fb5114e0f
Author: Jiajie Zhong <zh...@hotmail.com>
AuthorDate: Thu Jan 20 18:12:10 2022 +0800
[community] Add how to review issue or PR (#648)
---
community/en-us/join/review.md | 151 +++++++++++++++++++++++++++++++++++++++++
community/en-us/team.md | 83 +++++++++++-----------
community/zh-cn/join/review.md | 139 +++++++++++++++++++++++++++++++++++++
community/zh-cn/team.md | 82 +++++++++++-----------
site_config/community.jsx | 18 +++++
5 files changed, 388 insertions(+), 85 deletions(-)
diff --git a/community/en-us/join/review.md b/community/en-us/join/review.md
new file mode 100644
index 0000000..794bca8
--- /dev/null
+++ b/community/en-us/join/review.md
@@ -0,0 +1,151 @@
+# Community Review
+
+Beside submit Issues and pull requests to the GitHub repository mentioned in [team](../team.md), another important way to
+contribute to DolphinScheduler is reviewing GitHub Issues or Pull Requests. You can not only know the latest new and
+direction of the community, but also understand the good design in others during you reviewing. At the same time, you can
+increase your exposure in the community and accumulate your honor.
+
+Anyone is encouraged to review Issues and Pull Requests. We also raise a Help Wanted email discussion to solicit contributors
+from the community to review them. You could see detail in [mail][mail-review-wanted], we put the results of mail thread
+in [GitHub Discussion][discussion-result-review-wanted].
+
+> Note: It is only users mentioned in the [GitHub Discussion][discussion-result-review-wanted] can review Issues or Pull
+> Requests, Community advocates **Anyone is encouraged to review Issues and Pull Requests**. Users in
+> [GitHub Discussion][discussion-result-review-wanted] show their willing to review when we collect in the mail thread.
+> The advantage of this list is when the community has discussion, in addition to the mention Members in [team](../team.md),
+> you can also find some help in [GitHub Discussion][discussion-result-review-wanted] people. If you want to join the
+> [GitHub Discussion][discussion-result-review-wanted], please comment in that discussion and leave a module you are interested
+> in, and the maintainer will add you to the list.
+
+## How Reviewing
+
+DolphinScheduler receives community contributions through GitHub, and all its Issues and Pull Requests are hosted in GitHub.
+If you want to join community by reviewing, please go to section [review Issues](#issues), if you prefer Pull Requests please
+go to section [review Pull Requests](#pull-requests).
+
+### Issues
+
+Review Issues means discuss [Issues][all-issues] in GitHub and give suggestions on it. Include but are not limited to the following situations
+
+| Situation | Reason | Label | Action |
+| ------ | ------ | ------ | ------ |
+| wont fix | Has been fixed in dev branch | [wontfix][label-wontfix] | Close Issue, inform creator the fixed version if it already release |
+| duplicate issue | Had the same problem before | [duplicate][label-duplicate] | Close issue, inform creator the link of same issue |
+| Description not clearly | Without detail reproduce step | [need more information][label-need-more-information] | Inform creator add more description |
+
+In addition give suggestion, add label for issue is also important during review. The labeled issues can be retrieved
+better, which convenient for further processing. An issue can with more than one label. Common issue categories are:
+
+| Label | Meaning |
+| ------ | ------ |
+| [UI][label-UI] | UI and front-end related |
+| [security][label-security] | Security Issue |
+| [user experience][label-user-experience] | User experience Issue |
+| [development][label-development] | Development Issue |
+| [Python][label-Python] | Python Issue |
+| [plug-in][label-plug-in] | Plug-in Issue |
+| [document][label-document] | Document Issue |
+| [docker][label-docker] | Docker Issue |
+| [need verify][label-need-verify] | Need verify Issue |
+| [e2e][label-e2e] | E2E Issue |
+| [win-os][label-win-os] | windows operating system Issue |
+| [suggestion][label-suggestion] | Give suggestion to us |
+
+Beside classification, label could also set the priority of Issues. The higher the priority, the more attention pay
+in the community, the easier it is to be fixed or implemented. The priority label are as follows
+
+| Label | priority |
+| ------ | ------ |
+| [priority:high][label-priority-high] | High priority |
+| [priority:middle][label-priority-middle] | Middle priority |
+| [priority:low][label-priority-low] | Low priority |
+
+All the labels above in common label. For all labels in this project you could see in [full label list][label-all-list]
+
+Before reading following content, please make sure you have labeled the Issue.
+
+* Remove label [Waiting for reply][label-waiting-for-reply] after replying: Label [Waiting for reply][label-waiting-for-reply]
+ added when [creating an Issue][issue-choose]. It makes positioning un reply issue more convenient, and you should remove
+ this label after you reviewed it. If you do not remove it, will cause others to waste time looking on the same issue.
+* Mark [Waiting for review][label-waiting-for-review] when not sure whether issue is resolved or not: There are two situations
+ when you review issue. One is the problem has been located or resolved, maybe have to [Create PR](../development/submit-code.md)
+ when necessary. Secondly, you are not sure about this issue, you can labeled [Waiting for review][label-waiting-for-review]
+ and mention others to make a second confirmation.
+
+When an Issue need to create Pull Requests, you could also labeled it from below.
+
+| Label | Mean |
+| ------ | ------ |
+| [Chore][label-Chore] | Chore for project |
+| [Good first issue][label-good-first-issue] | Good first issue for new contributor |
+| [easy to fix][label-easy-to-fix] | Easy to fix, harder than `Good first issue` |
+| [help wanted][label-help-wanted] | Help wanted |
+
+> Note: Only members have permission to add or delete label. When you need to add or remove lebals but are not member,
+> you can `@` members to do that. But as long as you have a GitHub account, you can comment on issues and give suggestions.
+> We encourage everyone in the community to comment and answer issues
+
+### Pull Requests
+
+Review Pull mean discussing in [Pull Requests][all-PRs] in GitHub and giving suggestions to it. DolphinScheduler's
+Pull Requests reviewing are the same as [GitHub's reviewing changes in pull requests][gh-review-pr]. You can give your
+suggestions in Pull Requests
+
+* When you think the Pull Request is OK to be merged, you can agree to the Pull Request according to the "Approve" process
+ in [GitHub's reviewing changes in pull requests][gh-review-pr].
+* When you think Pull Request needs to be changed, you can comment it according to the "Comment" process in
+ [GitHub's reviewing changes in pull requests][gh-review-pr]. And when you think issues that must be fixed before they
+ merged, please follow "Request changes" in [GitHub's reviewing changes in pull requests][gh-review-pr] to ask contributors
+ modify it.
+
+Labeled Pull Requests is an important part. Reasonable classification can save a lot of time for reviewers. The good news
+is that the label's name and usage of Pull Requests are the same in [Issues](#issues), which can reduce the memory. For
+example, if there is a Pull Request is related to docker and block deployment. We can label it with [docker][label-docker]
+and [priority:high][label-priority-high].
+
+Pull Requests have some unique labels of it own
+
+| Label | Mean |
+| ------ | ------ |
+| [miss document][label-miss-document] | Pull Requests miss document, and should be add |
+| [first time contributor][label-first-time-contributor] | Pull Requests submit by first time contributor |
+| [don't merge][label-do-not-merge] | Pull Requests have some problem and should not be merged |
+
+> Note: Only members have permission to add or delete label. When you need to add or remove lebals but are not member,
+> you can `@` members to do that. But as long as you have a GitHub account, you can comment on Pull Requests and give suggestions.
+> We encourage everyone in the community to review Pull Requests
+
+[mail-review-wanted]: https://lists.apache.org/thread/9flwlzrp69xjn6v8tdkbytq8glqp2k51
+[discussion-result-review-wanted]: https://github.com/apache/dolphinscheduler/discussions/7545
+[label-wontfix]: https://github.com/apache/dolphinscheduler/labels/wontfix
+[label-duplicate]: https://github.com/apache/dolphinscheduler/labels/duplicate
+[label-need-more-information]: https://github.com/apache/dolphinscheduler/labels/need%20more%20information
+[label-win-os]: https://github.com/apache/dolphinscheduler/labels/win-os
+[label-waiting-for-reply]: https://github.com/apache/dolphinscheduler/labels/Waiting%20for%20reply
+[label-waiting-for-review]: https://github.com/apache/dolphinscheduler/labels/Waiting%20for%20review
+[label-user-experience]: https://github.com/apache/dolphinscheduler/labels/user%20experience
+[label-development]: https://github.com/apache/dolphinscheduler/labels/development
+[label-UI]: https://github.com/apache/dolphinscheduler/labels/UI
+[label-suggestion]: https://github.com/apache/dolphinscheduler/labels/suggestion
+[label-security]: https://github.com/apache/dolphinscheduler/labels/security
+[label-Python]: https://github.com/apache/dolphinscheduler/labels/Python
+[label-plug-in]: https://github.com/apache/dolphinscheduler/labels/plug-in
+[label-document]: https://github.com/apache/dolphinscheduler/labels/document
+[label-docker]: https://github.com/apache/dolphinscheduler/labels/docker
+[label-all-list]: https://github.com/apache/dolphinscheduler/labels
+[label-Chore]: https://github.com/apache/dolphinscheduler/labels/Chore
+[label-good-first-issue]: https://github.com/apache/dolphinscheduler/labels/good%20first%20issue
+[label-help-wanted]: https://github.com/apache/dolphinscheduler/labels/help%20wanted
+[label-easy-to-fix]: https://github.com/apache/dolphinscheduler/labels/easy%20to%20fix
+[label-priority-high]: https://github.com/apache/dolphinscheduler/labels/priority%3Ahigh
+[label-priority-middle]: https://github.com/apache/dolphinscheduler/labels/priority%3Amiddle
+[label-priority-low]: https://github.com/apache/dolphinscheduler/labels/priority%3Alow
+[label-miss-document]: https://github.com/apache/dolphinscheduler/labels/miss%20document
+[label-first-time-contributor]: https://github.com/apache/dolphinscheduler/labels/first%20time%20contributor
+[label-do-not-merge]: https://github.com/apache/dolphinscheduler/labels/don%27t%20merge
+[label-e2e]: https://github.com/apache/dolphinscheduler/labels/e2e
+[label-need-verify]: https://github.com/apache/dolphinscheduler/labels/need%20to%20verify
+[issue-choose]: https://github.com/apache/dolphinscheduler/issues/new/choose
+[all-issues]: https://github.com/apache/dolphinscheduler/issues
+[all-PRs]: https://github.com/apache/dolphinscheduler/pulls
+[gh-review-pr]: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/about-pull-request-reviews
diff --git a/community/en-us/team.md b/community/en-us/team.md
index e533c3b..1074b16 100644
--- a/community/en-us/team.md
+++ b/community/en-us/team.md
@@ -4,52 +4,49 @@ The DolphinScheduler team is comprised of Members and Contributors. Members have
## Members
-### PMC
-
-| Public Name | Apache ID | GitHub ID |
-|-------------------| ------------ | -------------- |
-| Furkan Kamaci | kamaci | kamaci |
-| Gang Li | lgcareer | lgcareer |
-| Guo Jiwei | technoboy | Technoboy- |
-| Guo William | guowei | William-GuoWei |
-| Jun Gao | gaojun2048 | gaojun2048 |
-| Kevin Ratnasekera | djkevincr | djkevincr |
-| Leon Bao | leonbao | lenboo,leonbao |
-| Liang Chen | chenliang613 | chenliang613 |
-| Lidong Dai | lidongdai | dailidong |
-| Qiao Zhanwei | journey | qiaozhanwei |
-| Shao Feng Shi | shaofengshi | shaofengshi |
-| Sheng Wu | wusheng | wu-sheng |
-| Wu Baoqi | baoqiwu | Baoqi |
-| Xiaochun Liu | liuxiaochun | khadgarmage |
-| ZijJian Gong | millionfor | millionfor |
-| Calvin Kirs | kirs | CalvinKirs |
-
+| Public Name | Apache ID | GitHub ID |
+|-------------------| ------------ | ---------------------------------------------------- |
+| Furkan Kamaci | kamaci | [@kamaci](https://github.com/kamaci) |
+| Gang Li | lgcareer | [@lgcareer](https://github.com/lgcareer) |
+| Guo Jiwei | technoboy | [@Technoboy-](https://github.com/Technoboy-) |
+| Guo William | guowei | [@William-GuoWei](https://github.com/William-GuoWei) |
+| Jun Gao | gaojun2048 | [@gaojun2048](https://github.com/gaojun2048) |
+| Kevin Ratnasekera | djkevincr | [@djkevincr](https://github.com/djkevincr) |
+| Leon Bao | leonbao | [@lenboo](https://github.com/lenboo) |
+| Liang Chen | chenliang613 | [@chenliang613](https://github.com/chenliang613) |
+| Lidong Dai | lidongdai | [@dailidong](https://github.com/dailidong) |
+| Qiao Zhanwei | journey | [@qiaozhanwei](https://github.com/qiaozhanwei) |
+| Shao Feng Shi | shaofengshi | [@shaofengshi](https://github.com/shaofengshi) |
+| Sheng Wu | wusheng | [@wu-sheng](https://github.com/wu-sheng) |
+| Wu Baoqi | baoqiwu | [@Baoqi](https://github.com/Baoqi) |
+| Xiaochun Liu | liuxiaochun | [@khadgarmage](https://github.com/khadgarmage) |
+| ZijJian Gong | millionfor | [@millionfor](https://github.com/millionfor) |
+| Calvin Kirs | kirs | [@CalvinKirs](https://github.com/CalvinKirs) |
### Committer
-| Public Name | Apache ID | GitHub ID |
-|---------------|--------------|-------------------|
-| BoYi Zhang | zhangboyi | BoYiZhang |
-| Caibiao Xiang | break60 | break60 |
-| Chong Zhuang | kerwin | zhuangchong |
-| Cong Huang | chongchongzi | chongchongzi |
-| Hemin Wen | wenhemin | wen-hemin |
-| Kai Zhu | zhukai | nauu |
-| Kejia Chen | jave | Jave-Chen |
-| Li Huang | eights | Eights-Li |
-| Lin Li | lilin | samz406 |
-| Shang Lou | clay4megtr | clay4444 |
-| Shaojie Wu | gabrywu | gabrywu |
-| Shiwen Cheng | chengshiwen | chengshiwen |
-| Wenhe Li | liwenhe | liwenhe1993 |
-| Xingchun Chen | chenxingchun | xingchun-chen |
-| Yichao Yang | yangyichao | yangyichao-mango |
-| Zongyao Zhang | lfyee | lfyee |
-| Zhenxu Ke | kezhenxu94 | kezhenxu94 |
-| Jiajie Zhong | zhongjiajie | zhongjiajie |
-| Yizhi Wang | wangyizhi | Wangyizhi1 |
-| ShunFeng Cai | caishunfeng | caishunfeng |
+| Public Name | Apache ID | GitHub ID |
+| ------------- | ------------ | -------------------------------------------------------- |
+| BoYi Zhang | zhangboyi | [@BoYiZhang](https://github.com/BoYiZhang) |
+| Caibiao Xiang | break60 | [@break60](https://github.com/break60) |
+| Chong Zhuang | kerwin | [@zhuangchong](https://github.com/zhuangchong) |
+| Cong Huang | chongchongzi | [@chongchongzi](https://github.com/chongchongzi) |
+| Hemin Wen | wenhemin | [@wen-hemin](https://github.com/wen-hemin) |
+| Kai Zhu | zhukai | [@nauu](https://github.com/nauu) |
+| Kejia Chen | jave | [@Jave-Chen](https://github.com/Jave-Chen) |
+| Li Huang | eights | [@Eights-Li](https://github.com/Eights-Li) |
+| Lin Li | lilin | [@samz406](https://github.com/samz406) |
+| Shang Lou | clay4megtr | [@clay4444](https://github.com/clay4444) |
+| Shaojie Wu | gabrywu | [@gabrywu](https://github.com/gabrywu) |
+| Shiwen Cheng | chengshiwen | [@chengshiwen](https://github.com/chengshiwen) |
+| Wenhe Li | liwenhe | [@liwenhe1993](https://github.com/liwenhe1993) |
+| Xingchun Chen | chenxingchun | [@xingchun-chen](https://github.com/xingchun-chen) |
+| Yichao Yang | yangyichao | [@yangyichao-mango](https://github.com/yangyichao-mango) |
+| Zongyao Zhang | lfyee | [@lfyee](https://github.com/lfyee) |
+| Zhenxu Ke | kezhenxu94 | [@kezhenxu94](https://github.com/kezhenxu94) |
+| Jiajie Zhong | zhongjiajie | [@zhongjiajie](https://github.com/zhongjiajie) |
+| Yizhi Wang | wangyizhi | [@Wangyizhi1](https://github.com/Wangyizhi1) |
+| ShunFeng Cai | caishunfeng | [@caishunfeng](https://github.com/caishunfeng) |
## Contributors
diff --git a/community/zh-cn/join/review.md b/community/zh-cn/join/review.md
new file mode 100644
index 0000000..5690873
--- /dev/null
+++ b/community/zh-cn/join/review.md
@@ -0,0 +1,139 @@
+# 参与社区 review
+
+贡献 DolphinScheduler 的方式,除了向 [团队](../team.md) 中提到的 GitHub 仓库提交 Issues 和 pull requests 外,另一非常重要的方式是
+review 社区的 Issues 或者 Pull Requests。通过别人 Issues 和 Pull Requests,你不仅能知道社区的最新进展和发展方向,还能了解别人代码的设
+计思想,同时可以增加自己在社区的曝光、积累自己在社区的荣誉值。
+
+任何人都被鼓励去 review 社区的 Issues 和 Pull Requests。我们还曾经发起过一个 Help Wanted 的邮件讨论,向社区征求贡献者协助 review Issues
+以及 Pull Requests,详见 [邮件][mail-review-wanted],并将其结果放到了 [GitHub Discussion][discussion-result-review-wanted] 中。
+
+> 注意: 这里并不是说只有 [GitHub Discussion][discussion-result-review-wanted] 中提及的用户才可以协助 review Issue 或者 Pull Requests,
+> 请记住社区的主张是 **任何人都被鼓励去 review 社区的 Issues 和 Pull Requests**。只是那部分用户在邮件列表意见征集的时候,表达了愿意付
+> 出更多的时间,参与社区的 review。另一个好处是,当社区有不确定的问题的时,除了可以找 [团队](../team.md) 中对应的 Members 外,还可以找
+> [GitHub Discussion][discussion-result-review-wanted] 中提及的人解答对应的问题。如果你要想要加入到 [GitHub Discussion][discussion-result-review-wanted]
+> 中,请在该 discussion 中评论并留下你感兴趣的模块,维护者会将你加入到对应的名单中。
+
+## 怎么参与社区 review
+
+DolphinScheduler 主要通过 GitHub 接收社区的贡献,其所有的 Issues 和 Pull Requests 都托管在 GitHub 中,如果你想参与 Issues 的 review
+具体请查看 [review Issues](#issues) 章节,如果你是想要参与 Pull Requests 的 review 具体请查看 [review Pull Requests](#pull-requests)
+章节。
+
+### Issues
+
+Review Issues 是指在 GitHub 中参与 [Issues][all-issues] 的讨论,并在对应的 Issues 给出建议。给出的建议包括但不限于如下的情况
+
+| 情况 | 原因 | 需增加标签 | 需要的动作 |
+| ------ | ------ | ------ | ------ |
+| 不需要修改 | 问题在 dev 分支最新代码中已经修复了 | [wontfix][label-wontfix] | 关闭 Issue,告知提出者将在那个版本发布,如已发布告知版本 |
+| 重复的问题 | 之前已经存在相同的问题 | [duplicate][label-duplicate] | 关闭 Issue,告知提出者相同问题的连接 |
+| 问题描述不清晰 | 没有明确说明问题如何复现 | [need more information][label-need-more-information] | 提醒用户需要增加缺失的描述 |
+
+除了个 issue 建议之外,给 Issue 分类也是非常重要的一个工作。分类后的 Issue 可以更好的被检索,为以后进一步处理提供便利。一个 Issue 可以被打上多个标签,常见的 Issue 分类有
+
+| 标签 | 标签代表的情况 |
+| ------ | ------ |
+| [UI][label-UI] | UI以及前端相关的 Issue |
+| [security][label-security] | 安全相关的 Issue |
+| [user experience][label-user-experience] | 用户体验相关的 Issue |
+| [development][label-development] | 开发者相关的 Issue |
+| [Python][label-Python] | Python相关的 Issue |
+| [plug-in][label-plug-in] | 插件相关的 Issue |
+| [document][label-document] | 文档相关的 Issue |
+| [docker][label-docker] | docker相关的 Issue |
+| [need verify][label-need-verify] | Issue 需要被验证 |
+| [e2e][label-e2e] | e2e相关的 Issue |
+| [win-os][label-win-os] | windows 操作系统相关的 Issue |
+| [suggestion][label-suggestion] | Issue 为项目提出了建议 |
+
+标签除了分类之外,还能区分 Issue 的优先级,优先级越高的标签越重要,越容易被重视,并会尽快被修复或者实现,优先级的标签如下
+
+| 标签 | 优先级 |
+| ------ | ------ |
+| [priority:high][label-priority-high] | 高优先级 |
+| [priority:middle][label-priority-middle] | 中优先级 |
+| [priority:low][label-priority-low] | 低优先级 |
+
+以上是常见的几个标签,更多的标签请查阅项目[全部的标签列表][label-all-list]
+
+在阅读以下内容是,请确保你已经为 Issue 打了标签。
+
+* 回复后及时去掉标签[Waiting for reply][label-waiting-for-reply]:在 [创建 Issue 的时候][issue-choose],我们会为 Issue 打上特定的标签
+ [Waiting for reply][label-waiting-for-reply],方便定位还没有被回复的 Issue,所以当你 review 了 Issue 之后,就需要将标签
+ [Waiting for reply][label-waiting-for-reply] 及时的从 Issue 中删除。
+* 打上 [Waiting for review][label-waiting-for-review] 标当你不确定这个 Issue 是否被解决:当你查阅了 Issue 后,会有两个情况出现。一是
+ 问题已经被定位或解决,如果创建 Pull Requests 的话,则参考 [创建PR](../development/submit-code.md)。二是你也不确定这个问题是否真的是
+ 被解决,这时你可以为 Issue 打上 [Waiting for review][label-waiting-for-review] 标签,并在 Issue 中 `@` 对应的人进行二次确认
+
+当 Issue 需要被创建 Pull Requests 解决,也可以视情况打上部分标签
+
+| 标签 | 标签代表的PR |
+| ------ | ------ |
+| [Chore][label-Chore] | 日常维护工作 |
+| [Good first issue][label-good-first-issue] | 适合首次贡献者解决的 Issue |
+| [easy to fix][label-easy-to-fix] | 比较容易解决 |
+| [help wanted][label-help-wanted] | 向社区寻求帮忙 |
+
+> 注意: 上面关于增加和删除标签的操作,目前只有成员可以操作,当你遇到需要增减标签的时候,但是不是成员是,可以 `@` 对应的成员让其帮忙增减。
+> 但只要你有 GitHub 账号就能评论 Issue,并给出建议。我们鼓励社区每人都去评论并为 Issue 给出解答
+
+### Pull Requests
+
+Review Pull 是指在 GitHub 中参与 [Pull Requests][all-PRs] 的讨论,并在对应的 Pull Requests 给出建议。DolphinScheduler review
+Pull Requests 与 [GitHub 的 reviewing changes in pull requests][gh-review-pr] 一样。你可以为 Pull Requests 提出自己的看法,
+
+* 当你认为这个 Pull Requests 没有问题,可以被合并的时候,可以根据 [GitHub 的 reviewing changes in pull requests][gh-review-pr] 的
+ approve 流程同意这个 Pull Requests。
+* 当你觉得这个 Pull Requests 需要被修改时,可以根据 [GitHub 的 reviewing changes in pull requests][gh-review-pr] 的 comment
+ 流程评论这个 Pull Requests。当你认为存在一定要先修复才能合并的问题,请参照 [GitHub 的 reviewing changes in pull requests][gh-review-pr]
+ 的 Request changes 流程要求贡献者修改 Pull Requests 的内容。
+
+为 Pull Requests 打上标签也是非常重要的一个环节,合理的分类能为后来的 reviewer 节省大量的时间。值得高兴的是,Pull Requests 的标签和 [Issues](#issues)
+中提及的标签和用法是一致的,这能减少 reviewer 对标签的记忆。例如这个 Pull Requests 是和 docker 并且直接影响到用户部署的,我们可以为他
+打上 [docker][label-docker] 和 [priority:high][label-priority-high] 的标签。
+
+除了和 Issue 类似的标签外,Pull Requests 还有许多自己特有的标签
+
+| 标签 | 含义 |
+| ------ | ------ |
+| [miss document][label-miss-document] | 该 Pull Requests 缺少文档 需要增加 |
+| [first time contributor][label-first-time-contributor] | 该 Pull Requests 贡献者是第一次贡献项目 |
+| [don't merge][label-do-not-merge] | 该 Pull Requests 有问题 暂时先不要合并 |
+
+> 注意: 上面关于增加和删除标签的操作,目前只有成员可以操作,当你遇到需要增减标签的时候,可以 `@` 对应的成员让其帮忙增减。但只要你有 GitHub
+> 账号就能评论 Pull Requests,并给出建议。我们鼓励社区每人都去评论并为 Pull Requests 给出建议
+
+[mail-review-wanted]: https://lists.apache.org/thread/9flwlzrp69xjn6v8tdkbytq8glqp2k51
+[discussion-result-review-wanted]: https://github.com/apache/dolphinscheduler/discussions/7545
+[label-wontfix]: https://github.com/apache/dolphinscheduler/labels/wontfix
+[label-duplicate]: https://github.com/apache/dolphinscheduler/labels/duplicate
+[label-need-more-information]: https://github.com/apache/dolphinscheduler/labels/need%20more%20information
+[label-win-os]: https://github.com/apache/dolphinscheduler/labels/win-os
+[label-waiting-for-reply]: https://github.com/apache/dolphinscheduler/labels/Waiting%20for%20reply
+[label-waiting-for-review]: https://github.com/apache/dolphinscheduler/labels/Waiting%20for%20review
+[label-user-experience]: https://github.com/apache/dolphinscheduler/labels/user%20experience
+[label-development]: https://github.com/apache/dolphinscheduler/labels/development
+[label-UI]: https://github.com/apache/dolphinscheduler/labels/UI
+[label-suggestion]: https://github.com/apache/dolphinscheduler/labels/suggestion
+[label-security]: https://github.com/apache/dolphinscheduler/labels/security
+[label-Python]: https://github.com/apache/dolphinscheduler/labels/Python
+[label-plug-in]: https://github.com/apache/dolphinscheduler/labels/plug-in
+[label-document]: https://github.com/apache/dolphinscheduler/labels/document
+[label-docker]: https://github.com/apache/dolphinscheduler/labels/docker
+[label-all-list]: https://github.com/apache/dolphinscheduler/labels
+[label-Chore]: https://github.com/apache/dolphinscheduler/labels/Chore
+[label-good-first-issue]: https://github.com/apache/dolphinscheduler/labels/good%20first%20issue
+[label-help-wanted]: https://github.com/apache/dolphinscheduler/labels/help%20wanted
+[label-easy-to-fix]: https://github.com/apache/dolphinscheduler/labels/easy%20to%20fix
+[label-priority-high]: https://github.com/apache/dolphinscheduler/labels/priority%3Ahigh
+[label-priority-middle]: https://github.com/apache/dolphinscheduler/labels/priority%3Amiddle
+[label-priority-low]: https://github.com/apache/dolphinscheduler/labels/priority%3Alow
+[label-miss-document]: https://github.com/apache/dolphinscheduler/labels/miss%20document
+[label-first-time-contributor]: https://github.com/apache/dolphinscheduler/labels/first%20time%20contributor
+[label-do-not-merge]: https://github.com/apache/dolphinscheduler/labels/don%27t%20merge
+[label-e2e]: https://github.com/apache/dolphinscheduler/labels/e2e
+[label-need-verify]: https://github.com/apache/dolphinscheduler/labels/need%20to%20verify
+[issue-choose]: https://github.com/apache/dolphinscheduler/issues/new/choose
+[all-issues]: https://github.com/apache/dolphinscheduler/issues
+[all-PRs]: https://github.com/apache/dolphinscheduler/pulls
+[gh-review-pr]: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/about-pull-request-reviews
diff --git a/community/zh-cn/team.md b/community/zh-cn/team.md
index faa6bea..e72313c 100644
--- a/community/zh-cn/team.md
+++ b/community/zh-cn/team.md
@@ -6,51 +6,49 @@ DolphinScheduler 团队由成员和贡献者组成。 成员可以直接访问 D
### PMC
-| Public Name | Apache ID | GitHub ID |
-|-------------------| ------------ | -------------- |
-| Furkan Kamaci | kamaci | kamaci |
-| Gang Li | lgcareer | lgcareer |
-| Guo Jiwei | technoboy | Technoboy- |
-| Guo William | guowei | William-GuoWei |
-| Jun Gao | gaojun2048 | gaojun2048 |
-| Kevin Ratnasekera | djkevincr | djkevincr |
-| Leon Bao | leonbao | lenboo,leonbao |
-| Liang Chen | chenliang613 | chenliang613 |
-| Lidong Dai | lidongdai | dailidong |
-| Qiao Zhanwei | journey | qiaozhanwei |
-| Shao Feng Shi | shaofengshi | shaofengshi |
-| Sheng Wu | wusheng | wu-sheng |
-| Wu Baoqi | baoqiwu | Baoqi |
-| Xiaochun Liu | liuxiaochun | khadgarmage |
-| ZijJian Gong | millionfor | millionfor |
-| Calvin Kirs | kirs | CalvinKirs |
-
+| Public Name | Apache ID | GitHub ID |
+|-------------------| ------------ | ---------------------------------------------------- |
+| Furkan Kamaci | kamaci | [@kamaci](https://github.com/kamaci) |
+| Gang Li | lgcareer | [@lgcareer](https://github.com/lgcareer) |
+| Guo Jiwei | technoboy | [@Technoboy-](https://github.com/Technoboy-) |
+| Guo William | guowei | [@William-GuoWei](https://github.com/William-GuoWei) |
+| Jun Gao | gaojun2048 | [@gaojun2048](https://github.com/gaojun2048) |
+| Kevin Ratnasekera | djkevincr | [@djkevincr](https://github.com/djkevincr) |
+| Leon Bao | leonbao | [@lenboo](https://github.com/lenboo) |
+| Liang Chen | chenliang613 | [@chenliang613](https://github.com/chenliang613) |
+| Lidong Dai | lidongdai | [@dailidong](https://github.com/dailidong) |
+| Qiao Zhanwei | journey | [@qiaozhanwei](https://github.com/qiaozhanwei) |
+| Shao Feng Shi | shaofengshi | [@shaofengshi](https://github.com/shaofengshi) |
+| Sheng Wu | wusheng | [@wu-sheng](https://github.com/wu-sheng) |
+| Wu Baoqi | baoqiwu | [@Baoqi](https://github.com/Baoqi) |
+| Xiaochun Liu | liuxiaochun | [@khadgarmage](https://github.com/khadgarmage) |
+| ZijJian Gong | millionfor | [@millionfor](https://github.com/millionfor) |
+| Calvin Kirs | kirs | [@CalvinKirs](https://github.com/CalvinKirs) |
### Committer
-| Public Name | Apache ID | GitHub ID |
-| ------------- | ------------ | ---------------- |
-| BoYi Zhang | zhangboyi | BoYiZhang |
-| Caibiao Xiang | break60 | break60 |
-| Chong Zhuang | kerwin | zhuangchong |
-| Cong Huang | chongchongzi | chongchongzi |
-| Hemin Wen | wenhemin | wen-hemin |
-| Kai Zhu | zhukai | nauu |
-| Kejia Chen | jave | Jave-Chen |
-| Li Huang | eights | Eights-Li |
-| Lin Li | lilin | samz406 |
-| Shang Lou | clay4megtr | clay4444 |
-| Shaojie Wu | gabrywu | gabrywu |
-| Shiwen Cheng | chengshiwen | chengshiwen |
-| Wenhe Li | liwenhe | liwenhe1993 |
-| Xingchun Chen | chenxingchun | xingchun-chen |
-| Yichao Yang | yangyichao | yangyichao-mango |
-| Zongyao Zhang | lfyee | lfyee |
-| Zhenxu Ke | kezhenxu94 | kezhenxu94 |
-| Jiajie Zhong | zhongjiajie | zhongjiajie |
-| Yizhi Wang | wangyizhi | Wangyizhi1 |
-| ShunFeng Cai | caishunfeng | caishunfeng |
-
+| Public Name | Apache ID | GitHub ID |
+| ------------- | ------------ | -------------------------------------------------------- |
+| BoYi Zhang | zhangboyi | [@BoYiZhang](https://github.com/BoYiZhang) |
+| Caibiao Xiang | break60 | [@break60](https://github.com/break60) |
+| Chong Zhuang | kerwin | [@zhuangchong](https://github.com/zhuangchong) |
+| Cong Huang | chongchongzi | [@chongchongzi](https://github.com/chongchongzi) |
+| Hemin Wen | wenhemin | [@wen-hemin](https://github.com/wen-hemin) |
+| Kai Zhu | zhukai | [@nauu](https://github.com/nauu) |
+| Kejia Chen | jave | [@Jave-Chen](https://github.com/Jave-Chen) |
+| Li Huang | eights | [@Eights-Li](https://github.com/Eights-Li) |
+| Lin Li | lilin | [@samz406](https://github.com/samz406) |
+| Shang Lou | clay4megtr | [@clay4444](https://github.com/clay4444) |
+| Shaojie Wu | gabrywu | [@gabrywu](https://github.com/gabrywu) |
+| Shiwen Cheng | chengshiwen | [@chengshiwen](https://github.com/chengshiwen) |
+| Wenhe Li | liwenhe | [@liwenhe1993](https://github.com/liwenhe1993) |
+| Xingchun Chen | chenxingchun | [@xingchun-chen](https://github.com/xingchun-chen) |
+| Yichao Yang | yangyichao | [@yangyichao-mango](https://github.com/yangyichao-mango) |
+| Zongyao Zhang | lfyee | [@lfyee](https://github.com/lfyee) |
+| Zhenxu Ke | kezhenxu94 | [@kezhenxu94](https://github.com/kezhenxu94) |
+| Jiajie Zhong | zhongjiajie | [@zhongjiajie](https://github.com/zhongjiajie) |
+| Yizhi Wang | wangyizhi | [@Wangyizhi1](https://github.com/Wangyizhi1) |
+| ShunFeng Cai | caishunfeng | [@caishunfeng](https://github.com/caishunfeng) |
## 贡献者
diff --git a/site_config/community.jsx b/site_config/community.jsx
index fc8b19f..758853d 100644
--- a/site_config/community.jsx
+++ b/site_config/community.jsx
@@ -40,6 +40,15 @@ export default {
],
},
{
+ title: 'Join Community',
+ children: [
+ {
+ title: 'Review Issue or Pull Requests',
+ link: '/en-us/community/join/review.html',
+ },
+ ],
+ },
+ {
title: 'Contribution Guide',
children: [
{
@@ -267,6 +276,15 @@ export default {
],
},
{
+ title: '参与社区',
+ children: [
+ {
+ title: 'Review Issue or Pull Requests',
+ link: '/zh-cn/community/join/review.html',
+ },
+ ],
+ },
+ {
title: '提交者指南',
children: [
{