You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by pa...@apache.org on 2022/11/06 11:43:53 UTC

[shardingsphere] branch master updated: Add more item for code conduct (#21978)

This is an automated email from the ASF dual-hosted git repository.

panjuan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
     new 2034c3b18c0 Add more item for code conduct (#21978)
2034c3b18c0 is described below

commit 2034c3b18c0a731284651ec1ca632d94bfba1689
Author: Liang Zhang <zh...@apache.org>
AuthorDate: Sun Nov 6 19:43:48 2022 +0800

    Add more item for code conduct (#21978)
---
 docs/community/content/involved/conduct/code.cn.md | 1 +
 docs/community/content/involved/conduct/code.en.md | 1 +
 2 files changed, 2 insertions(+)

diff --git a/docs/community/content/involved/conduct/code.cn.md b/docs/community/content/involved/conduct/code.cn.md
index 3bc69354bf3..150a398c15d 100644
--- a/docs/community/content/involved/conduct/code.cn.md
+++ b/docs/community/content/involved/conduct/code.cn.md
@@ -59,6 +59,7 @@ chapter = true
  - 条件运算符(<表达式1> ? <表达式2> : <表达式3>)禁止 `嵌套使用`。
  - 热点方法内应避免使用 Java Stream,除非该场景下使用 Stream 的性能优于普通循环。
  - 条件表达式中,优先使用正向语义,以便于理解代码逻辑。例如:`if (null == param) {} else {}`。
+ - 使用具体的 `@SuppressWarnings("xxx")` 代替 `@SuppressWarnings("all")`。
 
 ## 单元测试规范
 
diff --git a/docs/community/content/involved/conduct/code.en.md b/docs/community/content/involved/conduct/code.en.md
index 0f48b7cf9a7..4b3164e655c 100644
--- a/docs/community/content/involved/conduct/code.en.md
+++ b/docs/community/content/involved/conduct/code.en.md
@@ -56,6 +56,7 @@ The following code of conduct is based on full compliance with [ASF CODE OF COND
  - conditional operator(<expression1> ? <expression2> : <expression3>) `nested use` is forbidden.
  - Avoid using Java Stream in hot methods, unless the performance of using Stream is better than using loop in that situation.
  - Use forward semantics in priority for better understanding code logic in conditional expressions. For example: `if (null == param) {} else {}`.
+ - Use concentrate `@SuppressWarnings("xxx")` instead of `@SuppressWarnings("all")`.
 
 ## Contributor Covenant Unit Test of Conduct