uber/NullAway

Disallow writing `@Initializer` on constructors

Chiusa

#540 aperta il 6 gen 2022

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Java (288 fork)batch import
good first issue

Metriche repository

Star
 (3462 stelle)
Metriche merge PR
 (Merge medio 3g 23h) (14 PR mergiate in 30 g)

Descrizione

If a constructor is annotated with @Initializer, NullAway currently treats the constructor as if it were an initializer method that runs after any constructor, which clearly doesn't make sense. Instead, NullAway should report an error, as this is an invalid use of @Initializer.

Guida contributor

Direzione di ricerca
Look at NullAway's source code to understand how `@Initializer` is currently validated, likely in a handler or checker class. Examine existing checks for invalid annotation usage patterns, such as `@NonNull` on primitives. Add a new check that detects when `@Initializer` is applied to a constructor and reports an error. Consider writing a unit test to verify the behavior. Refer to issue #540 and the file `com/uber/nullaway/NullAway.java` or similar.
Tech stack
java
Dominio
backendtooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Javaannotation processing basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
55