liquid-java / liquid-java/liquidjava

Change `return` to `$result` for method return values in refinements

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

@rajshivu đang làm issue này rồi.

Từ ngày 9/2/2026.

  • #145 của @rajshivu — đang mở
enhancement good first issue
Ngôn ngữ chính
Java
Star
67
Fork
36
Merge trung bình
10 ngày 18 giờ
Pull request đã merge (30 ngày)
3

Mô tả

`return` is a statement, and it's strange to have it as a variable to represent the return value.
Therefore, we could change it for `result`. But since result can be an actual variable that represents something other than the return value we, we can try using `$result`.

This would mean a small change in the grammar, probably, and a change in replacing the return value.
Ps: the _ should still work in any case

Example Before:
```java
@Refinement("return > 0")
int test2 (){
return 10;
}
```
Example After:
```java
@Refinement("$result > 0")
int test2 (){
return 10;
}
```

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

Bắt đầu bằng cách xác định refinement grammar và phần mã thay thế hoặc kiểm tra các giá trị trả về; issue đề cập đến cả hai nhưng không nêu tên tệp hay test nào. So sánh cách xử lý hiện tại đối với `return` và `_` với cú pháp `$result` được đề xuất, đồng thời xem xét pull request đang mở trước khi bắt đầu. Công việc được xem là hoàn tất khi cú pháp mới hoạt động mà không làm hỏng hành vi hiện có.

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
Lĩnh vực
compilers
Loại issue
Tính năng
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
25/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.