From 7313b832b0950a190ba10f86968ac900968cd83c Mon Sep 17 00:00:00 2001 From: LamGC Date: Wed, 23 Sep 2020 12:55:56 +0800 Subject: [PATCH 01/10] =?UTF-8?q?[Add]=20Project=20=E6=B7=BB=E5=8A=A0?= =?UTF-8?q?=E8=B4=A1=E7=8C=AE=E6=8C=87=E5=8D=97;?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit [Add] CONTRIBUTING.md 添加一个尚未完成的贡献指南，完成后合并入 3.0.0 分支; --- .github/CONTRIBUTING.md | 38 ++++++++++++++++++++++++++++++++++++++ 1 file changed, 38 insertions(+) create mode 100644 .github/CONTRIBUTING.md diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md new file mode 100644 index 0000000..4bda5b7 --- /dev/null +++ b/.github/CONTRIBUTING.md @@ -0,0 +1,38 @@ +# 贡献指南 # + +欢迎你为本项目作出贡献！本项目的发展离不开所有贡献者的贡献。 +在为本项目作出任何贡献之前，请先阅读[贡献者准则](https://github.com/LamGC/ContentGrabbingJi/blob/3.0.0/CODE_OF_CONDUCT.md)， +一旦你尝试为本项目贡献，即代表你认同并接受贡献者准则，明白且接受因违反贡献者准则所造成的后果。 + +## 代码规范 ## +本项目代码规范大部分遵循 《Alibaba Java Coding Guidelines》， +详细内容可前往 https://github.com/alibaba/p3c 查阅。 +如果你的 IDE 为 Eclipse 或 Idea，则推荐你为 IDE 添加阿里巴巴根据该规范开发的规范检查插件，该插件有助于你保持代码规范。 + +### Commit Message ### +提交信息需要遵循以下格式： +``` +[TAG] ModuleName 对提交的说明; + +[TAG] FileName 对该文件的说明; +... + +``` +对于类代码文件而言，FileName只需要填写类名，对其他文件则需要包括后缀名。 +FileName 和 ModuleName 可按需要使用`, `追加，注意与说明空一格， +例如: `[Add][Change] SimpleClass, MainClass 添加 SimpleClass 并调整 MainClass 的日志格式;` +目前支持以下Tag: +- Add 文件添加 +- Change 文件更改 +- Update 更新文件 +- Delete 文件删除 +- Document 文档相关 +- Fix 问题修复 +- Optimize 优化相关 +- Move 文件移动(例如包更改) +- Rename 更改名称(类名, 文件名等) + +ModuleName 根据修改的相关子模块名填写即可，对项目而非子模块的更改，则 ModuleName 为 `Project`。 + +对大部分文件而言，FileName 按照以上规则填写即可，但对于依赖项而言，则要按照 GAV 坐标填写，例如：`[Add] junit:junit 添加 Junit 单元测试依赖项;` + From 26887d9d637b9c37ea708eb64c845b31aa9a83e6 Mon Sep 17 00:00:00 2001 From: LamGC Date: Sun, 18 Oct 2020 22:14:31 +0800 Subject: [PATCH 02/10] =?UTF-8?q?[Update]=20Project=20=E5=8F=88=E6=9B=B4?= =?UTF-8?q?=E6=96=B0=E4=B8=80=E7=82=B9=E7=82=B9=E8=B4=A1=E7=8C=AE=E6=8C=87?= =?UTF-8?q?=E5=8D=97(=E4=BE=9D=E7=84=B6=E6=98=AF=E6=9C=AA=E5=AE=8C?= =?UTF-8?q?=E6=88=90=E7=8A=B6=E6=80=81);?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .github/CONTRIBUTING.md | 82 ++++++++++++++++++++++++++++++++++------- 1 file changed, 68 insertions(+), 14 deletions(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index 4bda5b7..4a93ae5 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -1,8 +1,8 @@ # 贡献指南 # -欢迎你为本项目作出贡献！本项目的发展离不开所有贡献者的贡献。 +欢迎你为本项目作出贡献！本项目的发展离不开所有贡献者（不仅仅只是提交了相关代码，帮助编写文档，还是提出问题或想法也算是贡献者之一！）的贡献。 在为本项目作出任何贡献之前，请先阅读[贡献者准则](https://github.com/LamGC/ContentGrabbingJi/blob/3.0.0/CODE_OF_CONDUCT.md)， -一旦你尝试为本项目贡献，即代表你认同并接受贡献者准则，明白且接受因违反贡献者准则所造成的后果。 +一旦你尝试（或正在）为本项目贡献，即代表你认同并接受贡献者准则，清楚明白且接受因违反贡献者准则所造成的后果。 ## 代码规范 ## 本项目代码规范大部分遵循 《Alibaba Java Coding Guidelines》， @@ -14,25 +14,79 @@ ``` [TAG] ModuleName 对提交的说明; -[TAG] FileName 对该文件的说明; +[TAG] FileName/ClassName 对该文件的说明; ... ``` -对于类代码文件而言，FileName只需要填写类名，对其他文件则需要包括后缀名。 +#### 标点符号的使用 #### +在提交信息中，符号使用英文符号。 +#### 文件名或类名的说明 #### +对于类代码文件而言，FileName 只需要填写类名，对其他文件则需要包括后缀名。 FileName 和 ModuleName 可按需要使用`, `追加，注意与说明空一格， 例如: `[Add][Change] SimpleClass, MainClass 添加 SimpleClass 并调整 MainClass 的日志格式;` + +#### 标签的使用 #### 目前支持以下Tag: -- Add 文件添加 -- Change 文件更改 -- Update 更新文件 -- Delete 文件删除 -- Document 文档相关 -- Fix 问题修复 -- Optimize 优化相关 -- Move 文件移动(例如包更改) -- Rename 更改名称(类名, 文件名等) +- `Add` 文件添加 +- `Change` 文件更改 +- `Update` 更新文件 +- `Delete` 文件删除 +- `Document` 文档相关 +- `Fix` 问题修复 +- `Optimize` 优化相关 +- `Move` 文件移动(例如包更改) +- `Rename` 更改名称(类名, 文件名等) +- `Issue:Id` 指定 Commit 所关联的 Github Issue（如果与其他标签有关，可直接加到指定标签后方，该标签可不用加） +- `PR:ID` 指定 Commit 所关联的 Pull Request（可选的，可直接使用 Github 的默认 Commit Message） ModuleName 根据修改的相关子模块名填写即可，对项目而非子模块的更改，则 ModuleName 为 `Project`。 -对大部分文件而言，FileName 按照以上规则填写即可，但对于依赖项而言，则要按照 GAV 坐标填写，例如：`[Add] junit:junit 添加 Junit 单元测试依赖项;` +对大部分文件而言，FileName 按照以上规则填写即可，但对于依赖项而言，则要按照 GAV 坐标填写， +例如：`[Add] junit:junit 添加 Junit 单元测试依赖项;`， +或者更新依赖项：`[Update] junit:junit 更新 Junit 以修复潜在的漏洞('4.13' -> '4.13.1');` + +#### 引用 Github 中的相关 Issue 或者 PR #### +如需引用相关 Issue 或者 PR（Pull Request），可在相关 Tag 中引用，或者另起一个 Tag 并声明指定的 ID，例如： +``` +[Fix #11] Module 修复了部分功能异常情况; + +... +``` +或者是： +``` +[Change][Issue #15] Module 增加了一些功能; + +... +``` +对于需要修复的相关 Issue，既可以采用第一种形式，也可以用第二种形式，单独添加一个 Issue 标签标明关联的 Issue 即可，比如这样： +``` +[Fix][Issue #17] Module 修复了返回异常的问题; + +... +``` +如确定修复完成，也可以直接使用 Github 会对其作出相关操作的词（？），比如： +``` +[Fix close#20] Module 修复响应码判断错误的问题; + +... +``` +或者是 +``` +[Fix][Issue fixed#28] Module 修复模板编译错误的问题; + +... +``` + + +## 代码质量 ## +如果你想添加新的功能，请务必对新功能相关代码编写好覆盖全，情况完善的单元测试！ +覆盖全，多角度，高度完善的单元测试是保证代码、项目质量必不可少的！ +每次提交后，请检查 Action，查看 Github 中的代码是否能通过所有的单元测试。 + + + + + + + From 1ac7507868889d3eeebcf2ed4c66bd18bc634379 Mon Sep 17 00:00:00 2001 From: LamGC Date: Sun, 18 Oct 2020 22:21:12 +0800 Subject: [PATCH 03/10] =?UTF-8?q?[Update]=20Project=20=E5=86=8D=E6=9B=B4?= =?UTF-8?q?=E6=96=B0=E4=B8=80=E7=82=B9=E7=82=B9=E8=B4=A1=E7=8C=AE=E8=80=85?= =?UTF-8?q?=E6=8C=87=E5=8D=97;?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit [Update] .github/CONTRIBUTING.md 稍微标注一下相关词汇; --- .github/CONTRIBUTING.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index 4a93ae5..7e4dc3c 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -2,7 +2,7 @@ 欢迎你为本项目作出贡献！本项目的发展离不开所有贡献者（不仅仅只是提交了相关代码，帮助编写文档，还是提出问题或想法也算是贡献者之一！）的贡献。 在为本项目作出任何贡献之前，请先阅读[贡献者准则](https://github.com/LamGC/ContentGrabbingJi/blob/3.0.0/CODE_OF_CONDUCT.md)， -一旦你尝试（或正在）为本项目贡献，即代表你认同并接受贡献者准则，清楚明白且接受因违反贡献者准则所造成的后果。 +一旦你尝试（或正在）为本项目贡献，即代表你**认同并接受**贡献者准则，**清楚明白且接受**因**违反贡献者准则**所造成的后果。 ## 代码规范 ## 本项目代码规范大部分遵循 《Alibaba Java Coding Guidelines》， From 21f5373231307963baebce08881eaae18eafdb45 Mon Sep 17 00:00:00 2001 From: LamGC Date: Sat, 24 Oct 2020 15:40:03 +0800 Subject: [PATCH 04/10] =?UTF-8?q?[Update]=20Project=20=E6=9B=B4=E6=96=B0?= =?UTF-8?q?=E8=B4=A1=E7=8C=AE=E6=8C=87=E5=8D=97=E7=9A=84=E4=BE=9D=E8=B5=96?= =?UTF-8?q?=E7=9B=B8=E5=85=B3=E8=AF=B4=E6=98=8E;?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit [Update] CONTRIBUTING.md 增加依赖相关部分, 调整其他内容的细节; --- .github/CONTRIBUTING.md | 27 ++++++++++++++++----------- 1 file changed, 16 insertions(+), 11 deletions(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index 7e4dc3c..09efebf 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -19,12 +19,19 @@ ``` #### 标点符号的使用 #### -在提交信息中，符号使用英文符号。 +在提交信息中，符号使用**英文符号**。 #### 文件名或类名的说明 #### 对于类代码文件而言，FileName 只需要填写类名，对其他文件则需要包括后缀名。 FileName 和 ModuleName 可按需要使用`, `追加，注意与说明空一格， 例如: `[Add][Change] SimpleClass, MainClass 添加 SimpleClass 并调整 MainClass 的日志格式;` +ModuleName 根据修改的相关子模块名填写即可，对项目而非子模块的更改，则 ModuleName 为 `Project`，一个例子： +``` +[Change][Update][Optimize] Project, SubModule01, WebFrontend 更新项目中的Readme, 修改 SubModule01 中鉴权的实现细节, 优化Web前端模块的启动速度; +[Update] Readme.md 调整文档格式; +[Change] AuthorizationChecker 调整鉴权过程以避免潜在的错误; +[Optimize] MainClass 优化初始化速度, 移除不必要的检查步骤; +``` #### 标签的使用 #### 目前支持以下Tag: - `Add` 文件添加 @@ -33,13 +40,13 @@ FileName 和 ModuleName 可按需要使用`, `追加，注意与说明空一格 - `Delete` 文件删除 - `Document` 文档相关 - `Fix` 问题修复 -- `Optimize` 优化相关 +- `Optimize` 优化相关(或者代码整理啥的) - `Move` 文件移动(例如包更改) - `Rename` 更改名称(类名, 文件名等) - `Issue:Id` 指定 Commit 所关联的 Github Issue（如果与其他标签有关，可直接加到指定标签后方，该标签可不用加） - `PR:ID` 指定 Commit 所关联的 Pull Request（可选的，可直接使用 Github 的默认 Commit Message） -ModuleName 根据修改的相关子模块名填写即可，对项目而非子模块的更改，则 ModuleName 为 `Project`。 + 对大部分文件而言，FileName 按照以上规则填写即可，但对于依赖项而言，则要按照 GAV 坐标填写， 例如：`[Add] junit:junit 添加 Junit 单元测试依赖项;`， @@ -79,14 +86,12 @@ ModuleName 根据修改的相关子模块名填写即可，对项目而非子模 ## 代码质量 ## -如果你想添加新的功能，请务必对新功能相关代码编写好覆盖全，情况完善的单元测试！ -覆盖全，多角度，高度完善的单元测试是保证代码、项目质量必不可少的！ +如果你想添加新的功能，请务必对新功能相关代码编写好覆盖全，情况完善的单元测试（尽可能的覆盖所有新增功能，确保对大部分情况都有足够的测试）！ +覆盖全、多角度、高度完善的单元测试是保证代码、项目质量必不可少的！ 每次提交后，请检查 Action，查看 Github 中的代码是否能通过所有的单元测试。 - - - - - - +## 依赖管理 ## +请**尽可能避免引入**新依赖项，如果某个依赖项的引入，仅仅是用于某些计算，则这个引用是完全没有必要的， +对于因极少需要而引入某些依赖项时，请考虑了解依赖项内的*实现细节*，并复现在项目中，以避免引入依赖项。（当然，这是有例外的，如果某个功能较为复杂，且在该依赖项拥有很好的质量保证时，可考虑引入，具体细节可发起**issue**与我们进行讨论。） +**另外！**引入依赖项请注意**License**，确保依赖项可以被本项目使用。 From ee4681cfe36f73277bb87859353cecc9f9373d40 Mon Sep 17 00:00:00 2001 From: LamGC Date: Sat, 24 Oct 2020 15:45:28 +0800 Subject: [PATCH 05/10] =?UTF-8?q?[Fix]=20Project=20=E4=BF=AE=E5=A4=8D?= =?UTF-8?q?=E8=B4=A1=E7=8C=AE=E6=8C=87=E5=8D=97=E4=B8=AD=E7=9A=84=E4=B8=80?= =?UTF-8?q?=E4=BA=9B=E7=BB=86=E8=8A=82=E9=94=99=E8=AF=AF;?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit [Fix] CONTRIBUTING.md 修复了一个换行错误，然后调整了一下格式; --- .github/CONTRIBUTING.md | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index 09efebf..1834789 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -91,7 +91,8 @@ ModuleName 根据修改的相关子模块名填写即可，对项目而非子模 每次提交后，请检查 Action，查看 Github 中的代码是否能通过所有的单元测试。 ## 依赖管理 ## -请**尽可能避免引入**新依赖项，如果某个依赖项的引入，仅仅是用于某些计算，则这个引用是完全没有必要的， -对于因极少需要而引入某些依赖项时，请考虑了解依赖项内的*实现细节*，并复现在项目中，以避免引入依赖项。（当然，这是有例外的，如果某个功能较为复杂，且在该依赖项拥有很好的质量保证时，可考虑引入，具体细节可发起**issue**与我们进行讨论。） +> 请**尽可能避免引入**新依赖项，如果某个依赖项的引入，仅仅是用于某些计算，则这个引用是完全没有必要的。 + +对于因极少需要而引入某些依赖项时，请考虑了解依赖项内的*实现细节*，并复现在项目中，以避免引入依赖项。（当然，这是有例外的，如果某个功能较为复杂，且在该依赖项拥有很好的质量保证时，可考虑引入，具体细节可发起**issue**与我们进行讨论。） **另外！**引入依赖项请注意**License**，确保依赖项可以被本项目使用。 From 0af064715305f906f637b91112ec65a60297941a Mon Sep 17 00:00:00 2001 From: LamGC Date: Sat, 24 Oct 2020 15:51:39 +0800 Subject: [PATCH 06/10] =?UTF-8?q?[Change]=20Project=20=E5=B0=86=E8=B4=A1?= =?UTF-8?q?=E7=8C=AE=E6=8C=87=E5=8D=97=E4=B8=AD=20'Commit=20Message'=20?= =?UTF-8?q?=E7=AB=A0=E8=8A=82=E5=8D=87=E7=BA=A7=E4=B8=BA=E4=BA=8C=E7=BA=A7?= =?UTF-8?q?=E6=A0=87=E9=A2=98;?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit [Change] CONTRIBUTING.md 将 'Commit Message' 章节标题升级为二级标题; --- .github/CONTRIBUTING.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index 1834789..e75eb65 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -9,7 +9,7 @@ 详细内容可前往 https://github.com/alibaba/p3c 查阅。 如果你的 IDE 为 Eclipse 或 Idea，则推荐你为 IDE 添加阿里巴巴根据该规范开发的规范检查插件，该插件有助于你保持代码规范。 -### Commit Message ### +## Commit Message ## 提交信息需要遵循以下格式： ``` [TAG] ModuleName 对提交的说明; From c029f149a416969ee879e7c39ce34fea221d1abf Mon Sep 17 00:00:00 2001 From: LamGC Date: Sun, 8 Nov 2020 20:32:29 +0800 Subject: [PATCH 07/10] =?UTF-8?q?[Update]=20Project=20=E4=B8=BA=E8=B4=A1?= =?UTF-8?q?=E7=8C=AE=E6=8C=87=E5=8D=97=E6=B7=BB=E5=8A=A0=E8=AD=A6=E5=91=8A?= =?UTF-8?q?=E8=AF=B4=E6=98=8E,=20=E4=BB=A5=E8=A1=A8=E6=98=8E=E8=AF=A5?= =?UTF-8?q?=E6=8C=87=E5=8D=97=E5=B0=9A=E6=9C=AA=E5=AE=8C=E6=88=90;?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit [Update] .github/CONTRIBUTING.md 添加警告说明以表明该指南尚未完成, 有未提及事项需联系项目相关管理者; --- .github/CONTRIBUTING.md | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index e75eb65..313baf2 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -1,5 +1,8 @@ # 贡献指南 # +> 警告: 该指南尚未完成，所以在遇到与贡献有关问题时应该及时联系项目管理者以确定有关贡献事项， +而不应该因该指南未提及有关事项而擅作主张，擅作主张将会在 Review 阶段造成不必要的麻烦（甚至会被拒绝）。 + 欢迎你为本项目作出贡献！本项目的发展离不开所有贡献者（不仅仅只是提交了相关代码，帮助编写文档，还是提出问题或想法也算是贡献者之一！）的贡献。 在为本项目作出任何贡献之前，请先阅读[贡献者准则](https://github.com/LamGC/ContentGrabbingJi/blob/3.0.0/CODE_OF_CONDUCT.md)， 一旦你尝试（或正在）为本项目贡献，即代表你**认同并接受**贡献者准则，**清楚明白且接受**因**违反贡献者准则**所造成的后果。 From 59095c67b5eac4018a4b4460ff272609626acd8a Mon Sep 17 00:00:00 2001 From: LamGC Date: Sun, 8 Nov 2020 20:36:06 +0800 Subject: [PATCH 08/10] =?UTF-8?q?[Update]=20Project=20=E5=AF=B9=E8=B4=A1?= =?UTF-8?q?=E7=8C=AE=E6=8C=87=E5=8D=97=E4=B8=AD=E9=92=88=E5=AF=B9=E6=8F=90?= =?UTF-8?q?=E4=BA=A4=E4=BF=A1=E6=81=AF=E9=87=8C=20GAV=20=E5=9D=90=E6=A0=87?= =?UTF-8?q?=E7=9A=84=E4=BD=BF=E7=94=A8=E8=AF=B4=E6=98=8E;?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit [Update] .github/CONTRIBUTING.md 补充对 Commit Message 中 GAV 坐标的使用说明; --- .github/CONTRIBUTING.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index 313baf2..8d1397a 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -51,9 +51,9 @@ ModuleName 根据修改的相关子模块名填写即可，对项目而非子模 -对大部分文件而言，FileName 按照以上规则填写即可，但对于依赖项而言，则要按照 GAV 坐标填写， +对大部分文件而言，FileName 按照以上规则填写即可，但对于依赖项而言，则要按照 GAV（GroupId:ArtifactId:Version）坐标去除 **Version** 项后填写， 例如：`[Add] junit:junit 添加 Junit 单元测试依赖项;`， -或者更新依赖项：`[Update] junit:junit 更新 Junit 以修复潜在的漏洞('4.13' -> '4.13.1');` +或者更新依赖项需要添加对依赖项版本号的变更：`[Update] junit:junit 更新 Junit 以修复潜在的漏洞('4.13' -> '4.13.1');` #### 引用 Github 中的相关 Issue 或者 PR #### 如需引用相关 Issue 或者 PR（Pull Request），可在相关 Tag 中引用，或者另起一个 Tag 并声明指定的 ID，例如： From 97d10fc1daccdf31379fc570498ecb599a42f9f7 Mon Sep 17 00:00:00 2001 From: LamGC Date: Sun, 8 Nov 2020 20:40:21 +0800 Subject: [PATCH 09/10] =?UTF-8?q?[Update]=20Project=20=E4=BF=AE=E5=A4=8D?= =?UTF-8?q?=E8=B4=A1=E7=8C=AE=E5=87=86=E5=88=99=E4=B8=AD=E7=9A=84=E6=A0=87?= =?UTF-8?q?=E7=82=B9=E7=AC=A6=E5=8F=B7=E9=94=99=E8=AF=AF=EF=BC=8C=E5=B9=B6?= =?UTF-8?q?=E8=A1=A5=E5=85=85=E8=AD=A6=E5=91=8A=E4=BF=A1=E6=81=AF;?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit [Update] .github/CONTRIBUTING.md 修复标点符号错误并补充警告信息细节; --- .github/CONTRIBUTING.md | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index 8d1397a..b09795d 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -1,7 +1,8 @@ # 贡献指南 # -> 警告: 该指南尚未完成，所以在遇到与贡献有关问题时应该及时联系项目管理者以确定有关贡献事项， -而不应该因该指南未提及有关事项而擅作主张，擅作主张将会在 Review 阶段造成不必要的麻烦（甚至会被拒绝）。 +> 警告：该指南尚未完成，所以在遇到与贡献有关问题时应该及时联系项目管理者以确定有关贡献事项准则， +如该指南未提及有关事项时，你不应该擅作主张地决定事项准则，擅作主张将会在 Review 阶段造成不必要的麻烦（甚至会被拒绝）。 +！！！这条消息将在贡献指南完善的差不多后去除。！！！ 欢迎你为本项目作出贡献！本项目的发展离不开所有贡献者（不仅仅只是提交了相关代码，帮助编写文档，还是提出问题或想法也算是贡献者之一！）的贡献。 在为本项目作出任何贡献之前，请先阅读[贡献者准则](https://github.com/LamGC/ContentGrabbingJi/blob/3.0.0/CODE_OF_CONDUCT.md)， From 7024510df90e38f36b06b957b67ea8128546c109 Mon Sep 17 00:00:00 2001 From: LamGC Date: Sun, 8 Nov 2020 20:57:28 +0800 Subject: [PATCH 10/10] =?UTF-8?q?[Update]=20Project=20=E4=BF=AE=E5=A4=8D?= =?UTF-8?q?=E8=B4=A1=E7=8C=AE=E6=8C=87=E5=8D=97=E4=B8=AD=E7=9A=84=E4=B8=80?= =?UTF-8?q?=E4=BA=9B=E7=BB=86=E8=8A=82=E9=97=AE=E9=A2=98,=20=E8=A1=A5?= =?UTF-8?q?=E5=85=85=E9=83=A8=E5=88=86=E5=86=85=E5=AE=B9=E7=9A=84=E7=BB=86?= =?UTF-8?q?=E8=8A=82;?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit [Update] .github/CONTRIBUTING.md 修复 Review 中指出的问题, 补充部分内容的细节; --- .github/CONTRIBUTING.md | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md index b09795d..81ffe0f 100644 --- a/.github/CONTRIBUTING.md +++ b/.github/CONTRIBUTING.md @@ -6,7 +6,7 @@ 欢迎你为本项目作出贡献！本项目的发展离不开所有贡献者（不仅仅只是提交了相关代码，帮助编写文档，还是提出问题或想法也算是贡献者之一！）的贡献。 在为本项目作出任何贡献之前，请先阅读[贡献者准则](https://github.com/LamGC/ContentGrabbingJi/blob/3.0.0/CODE_OF_CONDUCT.md)， -一旦你尝试（或正在）为本项目贡献，即代表你**认同并接受**贡献者准则，**清楚明白且接受**因**违反贡献者准则**所造成的后果。 +一旦你尝试（或正在）为本项目贡献（贡献中，或者贡献过），即代表你**认同并接受**贡献者准则，**清楚明白且接受**因**违反贡献者准则**所造成的后果。 ## 代码规范 ## 本项目代码规范大部分遵循 《Alibaba Java Coding Guidelines》， @@ -50,7 +50,7 @@ ModuleName 根据修改的相关子模块名填写即可，对项目而非子模 - `Issue:Id` 指定 Commit 所关联的 Github Issue（如果与其他标签有关，可直接加到指定标签后方，该标签可不用加） - `PR:ID` 指定 Commit 所关联的 Pull Request（可选的，可直接使用 Github 的默认 Commit Message） - +> 注意：后续如有需要，会追加一些标签。 对大部分文件而言，FileName 按照以上规则填写即可，但对于依赖项而言，则要按照 GAV（GroupId:ArtifactId:Version）坐标去除 **Version** 项后填写， 例如：`[Add] junit:junit 添加 Junit 单元测试依赖项;`， @@ -88,15 +88,16 @@ ModuleName 根据修改的相关子模块名填写即可，对项目而非子模 ... ``` - ## 代码质量 ## 如果你想添加新的功能，请务必对新功能相关代码编写好覆盖全，情况完善的单元测试（尽可能的覆盖所有新增功能，确保对大部分情况都有足够的测试）！ 覆盖全、多角度、高度完善的单元测试是保证代码、项目质量必不可少的！ 每次提交后，请检查 Action，查看 Github 中的代码是否能通过所有的单元测试。 ## 依赖管理 ## -> 请**尽可能避免引入**新依赖项，如果某个依赖项的引入，仅仅是用于某些计算，则这个引用是完全没有必要的。 +> 请**尽可能避免引入**新依赖项，如果某个依赖项的引入，仅仅是用于某些计算（或某个小功能，比如仅使用 Apache Commons 中的 `RandomUtils` 类获取 long 类型的随机数）， +则这个依赖项的引入是完全没有必要的。 -对于因极少需要而引入某些依赖项时，请考虑了解依赖项内的*实现细节*，并复现在项目中，以避免引入依赖项。（当然，这是有例外的，如果某个功能较为复杂，且在该依赖项拥有很好的质量保证时，可考虑引入，具体细节可发起**issue**与我们进行讨论。） +对于因极少需要而引入某些依赖项时，请考虑了解依赖项内的*实现细节*，并复现在项目中，以避免引入依赖项。 +（当然，这是有例外的，如果某个功能较为复杂，且在该依赖项拥有很好的质量保证时，可考虑引入，具体细节可发起**issue**与我们进行讨论。） **另外！**引入依赖项请注意**License**，确保依赖项可以被本项目使用。