redhat-developer / redhat-developer/vscode-java
Add support for @CheckReturnValue
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- TypeScript
- Estrellas
- 2.3k
- Forks
- 546
- Merge medio
- 20 h 1 min
- PR fusionados (30 d)
- 11
Descripción
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
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
No se nombran archivos fuente ni pruebas. Empieza por localizar el punto de entrada del diagnóstico o análisis de Java de VS Code y revisar cómo se gestionan las anotaciones de métodos; el trabajo está terminado cuando las anotaciones @CheckReturnValue independientes del paquete produzcan una advertencia para los resultados ignorados, con cobertura para llamadas reactivas, de streams o de estilo builder.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- java, typescript, vscode
- Área
- developer-experience, devtools
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 35/100