grpc / grpc/grpc-java

Try to remove SuppressWarnings("GuardedBy")

Offen
#6,578 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
code health
Vorherrschende Sprache
Java
Sterne
12.1k
Forks
4k
Ø Merge
2 T. 17 Std.
Gemergte PRs (30 T.)
37

Beschreibung

#6566 introduced suppressions to allow being compatible with newer versions of Error Prone. But we should spend some time to see if we could tweak the code so that the suppressions would be unnecessary.

If we can't remove them or if it would make the code worse, we can leave them as-is and close this issue.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Start by reviewing issue #6566 and the suppressions it introduced for newer Error Prone compatibility. Trace each SuppressWarnings("GuardedBy") use and assess whether the surrounding code can be adjusted without making it worse. Done means removing the suppressions where safe, or documenting that they should remain and closing the issue.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
backend
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.