errorprone cleanliness
- Linguagem predominante
- Java
- Estrelas
- 3.1k
- Forks
- 1.4k
- Merge médio
- 7d 14h
- PRs com merge (30d)
- 31
Descrição
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.
Guia de contribuição
Direção de pesquisa
Comece pela configuração do compilador Error Prone mostrada em pom.xml e revise o PR #12425 vinculado antes de iniciar. Execute o build do Maven para identificar os problemas reportados e, em seguida, acompanhe as correções individuais como issues filhas ou pull requests; o trabalho estará concluído quando os problemas reportados forem resolvidos e o Error Prone puder permanecer como um hook pre-commit.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- java
- Domínio
- build-system
- Tipo de issue
- Refatoração
- Dificuldade
- 5/5
- Tempo estimado
- Mais de uma semana
- Status de atividade
- Estagnada
- Clareza
- Precisa de esclarecimento
- Facilidade para iniciantes
- 25/100