Which version of "googleJavaFormat" to use?
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 25/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- java
- Lĩnh vực
- build-system
Hướng nghiên cứu
Bắt đầu bằng cách xem xét thiết lập hiện tại của spotless-maven-plugin và googleJavaFormat, sau đó chạy ./mvnw spotless:check với các phiên bản JDK liên quan. Công việc được coi là hoàn tất khi đã chọn và ghi lại chiến lược phiên bản được hỗ trợ, cập nhật cấu hình nếu cần, và xác nhận rằng các kiểm tra định dạng hoạt động với phạm vi JDK dự kiến.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Currently we are using spotless-maven-plugin:2.44.4 together with googleJavaFormat 1.19.1 to format the java sources. This does not work in JDK8 but all later versions.
With JDK25, spotless-maven-plugin:3.0.0 and googleJavaFormat 1.27.0 are the minimal versions. Older versions will simply fail when ./mvnw spotless:check is executed.
This version does require JDK17 or higher.
I see three options to continue:
a) Stick with 2.44.4 and 1.19.1 and do not run spotless in JDK25 and later.
b) Upgrade to 3.00 and 1.27.0 and do not run spotless in JDK16 and earlier.
c) Choose the spotless version depending on the JDK version.
a) Is least effort for existing developers, but makes onboarding for new contributors more complicated.
b) Is least effort for new developers, might be a problem if someone is required to use a JDK before 17.
c) At the moment, the differences in the formatters (mostly indenting in comments) have no impact on existing code. That might change with added documentation in the future, resulting in formatters changing back and forth.
My prefered solution is b). Are there any showstoppers?
Alternatively we could go for c) until we have see a problem and switch to b) then.
What are your thoughts?
- Ngôn ngữ chính
- Java
- Star
- 334
- Fork
- 319
- Merge trung bình
- 10 ngày 8 giờ
- Pull request đã merge (30 ngày)
- 2
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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.
Issue khác của cloudfoundry/cf-java-client
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
cloudfoundry/cf-java-client#1374 ·
-
Integration tests fail: test-service-broker.jar incompatible with java-buildpack v5 (JDK 17 default)Đang mở
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 74/100
cloudfoundry/cf-java-client#1344 · 1 bình luận ·
-
Execute integration tests in Github ActionsCó thể đã có người làm @ZPascal đã nhận 1 ngày trước. Đang mở
cloudfoundry/cf-java-client#1380 · 1 reaction · 2 người được giao ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
cloudfoundry/cf-java-client#1373 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
cloudfoundry/cf-java-client#1370 ·
Tất cả issue của cloudfoundry/cf-java-client
Issue tương tự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
liquid-java/liquidjava#321 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày
-
rename config parameterĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
OpenConext/OpenConext-access#1015 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
68000: FDBcc branch target is 2 bytes too low (uses inst_start+2 instead of the displacement word)Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
NationalSecurityAgency/ghidra#9702 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
apache/rocketmq-dashboard#5358 ·
Maintainer thường phản hồi trong vòng 3 ngày