apache / apache/cloudstack

errorprone cleanliness

Aperta
#11,438 0 commenti 0 reazioni 0 assegnatari Rivendicata da @Pearl1594 Vedi su GitHub
complexity:dragoning type:stability
Lingua principale
Java
Stelle
3.1k
Fork
1.4k
Merge medio
6g 19h
PR unite (30g)
32

Descrizione

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.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia con la configurazione del compilatore Error Prone mostrata in pom.xml ed esamina la PR #12425 collegata prima di iniziare. Esegui la build Maven per identificare i problemi segnalati, quindi monitora le singole correzioni come issue figlie o pull request; il lavoro è completato quando i problemi segnalati sono stati risolti ed Error Prone può rimanere un hook pre-commit.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
java
Ambito
build-system
Tipo di issue
Refactoring
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
25/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.