redhat-developer / redhat-developer/vscode-java
Add support for @CheckReturnValue
Personne n'a encore pris cette issue.
- Langage dominant
- TypeScript
- Étoiles
- 2.3k
- Forks
- 546
- Merge moyen
- 20 h 1 min
- PR mergées (30 j)
- 11
Description
IntelliJ has a "clever" way to inform the users that the result of a method must be used.
It detects that the method is annotated with a @CheckReturnValue annotation regardless of its package. So, frameworks can provide their own (like SmallRye Commons Annotations).
Basically, if you have an API with:
@CheckReturnValue
public Uni<String> doSomethingCool();
a user using that method like in the following snippet would get a warning:
doSomethingCool();
The warning indicates that the return value must be used, like in:
doSomethingCool().subscribe().with(r -> ...);
This is particularly interesting for reactive APIs, stream APIs, and builders APIs.
It would be great if VS Code Java could provide the same kind of feature.
CC @maxandersen
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Aucun fichier source ni test n’est nommé. Commence par localiser le point d’entrée du diagnostic ou de l’analyse Java de VS Code et examine comment les annotations de méthode sont traitées ; le travail est terminé lorsque les annotations @CheckReturnValue indépendantes du package produisent un avertissement pour les résultats ignorés, avec une couverture des appels réactifs, de stream ou de style builder.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- java, typescript, vscode
- Domaine
- developer-experience, devtools
- Type d'issue
- Fonctionnalité
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 35/100