uber/NullAway

Disallow writing `@Initializer` on constructors

Fermée

#540 ouverte le 6 janv. 2022

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)Java (288 forks)batch import
good first issue

Métriques du dépôt

Stars
 (3 462 étoiles)
Métriques de merge PR
 (Merge moyen 3j 23h) (14 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
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.
Stack technique
java
Domaine
backendtooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Javaannotation processing basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55