redhat-developer / redhat-developer/vscode-java

Add support for @CheckReturnValue

Đang mở
#2,238 0 bình luận 2 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

code analysis diagnostics enhancement
Ngôn ngữ chính
TypeScript
Star
2.3k
Fork
546
Merge trung bình
20 giờ 1 phút
Pull request đã merge (30 ngày)
11

Mô tả

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

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Không có tệp mã nguồn hoặc bài kiểm thử nào được nêu tên. Hãy bắt đầu bằng cách xác định điểm vào của chẩn đoán hoặc phân tích Java trong VS Code và xem xét cách các chú thích phương thức được xử lý; công việc hoàn tất khi các chú thích @CheckReturnValue không phụ thuộc vào package tạo ra cảnh báo cho các kết quả bị bỏ qua, với độ bao phủ cho các lệnh gọi reactive, stream hoặc theo kiểu builder.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
java, typescript, vscode
Lĩnh vực
developer-experience, devtools
Loại issue
Tính năng
Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Cần làm rõ
Mức phù hợp với người mới
35/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.