apache / apache/cloudstack

errorprone cleanliness

未关闭
#11,438 0 条评论 0 个 reaction 已指派 0 人 已被 @Pearl1594 认领 在 GitHub 查看
complexity:dragoning type:stability
主要语言
Java
星标
3.1k
派生
1.4k
平均合并
6 天 19 小时
30 天内合并 PR
32

描述

go through the code by instating error prone and fix any issues it reports. Once that epic work is done we can permanently install errorprone as a pre-commit hook.

```
commit 9f9a2b6b7e094537fea75f633d355f983d4c15d7
Author: Daan Hoogland
Date: Thu Jan 12 19:32:35 2023 +0100

errorprone

diff --git a/pom.xml b/pom.xml
index 204c74cc76..13c46d7521 100644
--- a/pom.xml
+++ b/pom.xml
@@ -817,6 +817,32 @@



+
+ org.apache.maven.plugins
+ maven-compiler-plugin
+ ${cs.compiler-plugin.version}
+
+ 8
+ 8
+ UTF-8
+
+ -XDcompilePolicy=simple
+ -Xplugin:ErrorProne
+
+
+
+ com.google.errorprone
+ error_prone_core
+ 2.18.0
+
+
+
+
+



@@ -1074,15 +1100,26 @@

${cs.jdk.version}
${cs.jdk.version}
- true
- 128m
- 512m
+ UTF-8

-XDignore.symbol.file=true
--add-opens=java.base/java.lang=ALL-UNNAMED
--add-exports=java.base/sun.security.x509=ALL-UNNAMED
--add-exports=java.base/sun.security.provider=ALL-UNNAMED
+ -XDcompilePolicy=simple
+ -Xplugin:ErrorProne
+ --illegal-access=warn

+
+
+ com.google.errorprone
+ error_prone_core
+ 2.18.0
+
+
+ true
+ 128m
+ 512m



```

the above can be added to the main pom.xml and for any issues reported child issues of this or PR can be created directly. The scope of this is would be enormous and too big to deal with at once.

贡献指南

打开贡献指南

调研方向

先从 pom.xml 中所示的 Error Prone 编译器配置开始,并在开始之前查看已链接的 PR #12425。运行 Maven 构建以识别报告的问题,然后将各项修复作为子 issue 或拉取请求进行跟踪;当报告的问题得到解决,并且 Error Prone 可以继续作为 pre-commit hook 时,这项工作就完成了。

由索引模型根据 Issue 内容生成。

评估

技术栈
java
领域
build-system
Issue 类型
重构
难度
5/5
预计耗时
一周以上
活跃度
停滞
描述清晰度
需要澄清
新手友好度
25/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。