Use Kotlin's documented Maven setup to compile Kotlin before Java
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 48/100
- Tipo de issue
- Refatoração
- Clareza
- Razoavelmente clara
- Status de atividade
- Ativa
- Domínio
- build-system
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- Kotlin
- Estrelas
- 824
- Forks
- 175
- Merge médio
- 2d 5h
- PRs com merge (30d)
- 52
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de graphql-java-kickstart/graphql-java-tools
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
graphql-java-kickstart/graphql-java-tools#893 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 72/100
graphql-java-kickstart/graphql-java-tools#889 ·
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 55/100
graphql-java-kickstart/graphql-java-tools#886 ·
Mantenedores costumam responder em até 1 dia
-
Reject directives applied at the wrong location, without a required argument or twice in 15.0.0Abertaenhancement
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 72/100
graphql-java-kickstart/graphql-java-tools#884 ·
Mantenedores costumam responder em até 1 dia
-
Directive wirings see a type reference on recursive input fieldsTalvez já em andamento @oryan-block assumiu há 1 dia. Abertabug
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 35/100
graphql-java-kickstart/graphql-java-tools#875 ·
Mantenedores costumam responder em até 1 dia
Todas as issues de graphql-java-kickstart/graphql-java-tools
Issues semelhantes
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
navikt/mock-oauth2-server#1129 ·
Mantenedores costumam responder em até 1 dia
-
Expose `close` on SsrEngineAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 77/100
-
unconfirmed
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
thunderbird/thunderbird-android#11778 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
world-federation-of-advertisers/cross-media-measurement#4709 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
Mantenedores costumam responder em até 1 dia