Temurin version incorrectly converted for setup-java 6.0.x
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- TypeScript
- Star
- 2k
- Fork
- 870
- Merge trung bình
- 3 giờ 16 phút
- Pull request đã merge (30 ngày)
- 11
Mô tả
Description:
After upgrade from 5.7.0 to 6.0.0 and 6.0.1, I noticed that setup-java does not recognize Temurin's versions anymore (e.g. "26.0.2+10"). This leads to an "is not valid SemVer notation for a Java version" error.
The following regular expression does not not pick up Temurin's version correctly:
The issue seems to be similar to: https://github.com/actions/setup-java/issues/1267
Task version:
v6, v6.0.1
Platform:
- Ubuntu
- macOS
- Windows
Runner type:
- Hosted
- Self-hosted
Repro steps:
- successful run for setup-java "5.7.0" and Temurin "26.0.2+10", "25.0.4+7"
- failed runs for setup-java "6.0.0" and Temurin "26.0.2.1+1", "25.0.4.1+1"
- failed runs for setup-java "6.0.1" and Temurin "26.0.2.1+1", "25.0.4.1+1"
Expected behavior:
Successfully resolve Temurin version.
Actual behavior:
setup-java >= 6.0.0 returns
The string '26.0.2.1+1' is not valid SemVer notation for a Java version. Please check README file for code snippets and more detailed information
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu tại src/distributions/base-installer.ts, khoảng dòng 681-689, nơi biểu thức chính quy xử lý các phiên bản Java. Tái hiện lỗi với các phiên bản Temurin như 26.0.2.1+1 và 25.0.4.1+1, sau đó xác minh rằng setup-java phân giải thành công các phiên bản này mà không gặp lỗi SemVer.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- github-actions, java, typescript
- Lĩnh vực
- ci-cd, devops
- Loại issue
- Lỗi
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức độ hoạt động
- Sôi nổi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức phù hợp với người mới
- 82/100