redhat-developer / redhat-developer/vscode-java
Add support for @CheckReturnValue
Nessuno ha ancora preso questa issue.
- Lingua principale
- TypeScript
- Stelle
- 2.3k
- Fork
- 546
- Merge medio
- 20h 1m
- PR unite (30g)
- 11
Descrizione
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
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Non sono indicati file sorgente o test. Inizia individuando il punto di ingresso della diagnostica o dell’analisi Java di VS Code e verifica come vengono gestite le annotazioni dei metodi; il lavoro è completato quando le annotazioni @CheckReturnValue indipendenti dal package producono un avviso per i risultati ignorati, con copertura per le chiamate reattive, stream o in stile builder.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- java, typescript, vscode
- Ambito
- developer-experience, devtools
- Tipo di issue
- Funzionalità
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 35/100