Use Kotlin's documented Maven setup to compile Kotlin before Java
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 48/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Lĩnh vực
- build-system
Hướng nghiên cứu
Read the Maven configuration for kotlin-maven-plugin and maven-compiler-plugin, including default-testCompile and the java17-tests profile. Compare it with the Kotlin Maven guide's second setup option, then check how module-info.java and the release profile's source and javadoc jars depend on the current executions. Done means Kotlin and Java compile in the required order across these configurations with no behavior change.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
We get Kotlin compiled before javac by binding the Kotlin executions to earlier phases: compile runs at process-sources, and since #879 test-compile runs at process-test-sources. #879 needed that so ResolverMethodsTest.java could call the Kotlin assertEveryFieldResolves helper. Without it, javac compiled the Java tests before any Kotlin test class existed.
It works, but it isn't how Kotlin's docs say to do it. The Maven guide has two options:
<extensions>true</extensions>onkotlin-maven-plugin, declared beforemaven-compiler-plugin. It adds the Kotlin executions itself, and the docs say not to configure the compiler plugin's executions with it.- Keep the Kotlin executions at
compile/test-compile, setdefault-compileanddefault-testCompileto<phase>none</phase>, and addjava-compile/java-test-compileexecutions after Kotlin.
The second one probably fits better, since we already configure default-testCompile (useModulePath=false, with a TODO for 15.0.0) and the java17-tests profile adds its own java17-test-compile execution. Both have to keep working, along with module-info.java and the release profile's source and javadoc jars.
No behaviour change, it's build cleanup only.
- Ngôn ngữ chính
- Kotlin
- Star
- 824
- Fork
- 175
- Merge trung bình
- 2 ngày 1 giờ
- Pull request đã merge (30 ngày)
- 59
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 graphql-java-kickstart/graphql-java-tools
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
graphql-java-kickstart/graphql-java-tools#893 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
graphql-java-kickstart/graphql-java-tools#905 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
graphql-java-kickstart/graphql-java-tools#900 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 72/100
graphql-java-kickstart/graphql-java-tools#889 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Reject directives applied at the wrong location, without a required argument or twice in 15.0.0Đang mởenhancement
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 72/100
graphql-java-kickstart/graphql-java-tools#884 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của graphql-java-kickstart/graphql-java-tools
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Kotlin/kotlinx-io#529 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
mcnaveen/health-connect-webhook#110 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Delete all aboveĐang mởenhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
mjaakko/NeoStumbler#1307 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
No Wallpaper: Black and white theme changes makes fonts look weird on Home screen and App listĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
tanujnotes/Olauncher#770 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100