Update kotlin to 1.9.10

[sdelamo]

No description provided.

[sdelamo]

@timyates see if you can make it pass.

[timyates]

Starting point https://ge.micronaut.io/s/ffg3lcvxu5vvy

:inject-kotlin-test:compileKotlin FAILED
:test-suite-kotlin:kaptTestKotlin FAILED
:context-propagation:compileTestKotlin FAILED

[timyates]

Ok, one failure

https://ge.micronaut.io/s/a4hp6gehhurja/tests/overview?outcome=FAILED

Hopefully fixed by 040041a

[timyates]

@melix Any idea why we're seeing

The Kotlin Gradle plugin was loaded multiple times in different subprojects, which is not supported and may break the build. 	
This might happen in subprojects that apply the Kotlin plugins with the Gradle 'plugins { ... }' DSL if they specify explicit versions, even if the versions are equal.	
Please add the Kotlin plugin to the common parent project or the root project, then remove the versions in the subprojects.	
If the parent project does not need the plugin, add 'apply false' to the plugin line.	
See: https://docs.gradle.org/current/userguide/plugins.html#sec:subprojects_plugins_dsl	
The Kotlin plugin was loaded in the following projects: ':aop', ':inject-kotlin'

https://ge.micronaut.io/s/x33xdbydcmbke/console-log?page=1#L24

What it says doesn't seem to be the case 🤔

Edit

Is this the fix? 6b28a0f

[dstepanov]

You need to update gradle.properties

[timyates]

You need to update gradle.properties

Which bit? I've already bumped the kotlin version 🤔

[dstepanov]

The bit that says kotlinVersion

[timyates]

@dstepanov I already had done in f33adbf

And now I'm seeing if it works with it removed in 6b28a0f

[dstepanov]

Sorry I didn't see it in diff

[timyates]

Going to investigate the deprecations

https://ge.micronaut.io/s/xwtd572yhxyby/console-log?page=1#L149

later...

So it looks like the ComponentRegistrar deprecation/removal has been delayed so I guess we can ignore for now.

[timyates]

This test no longer throws an exception. I am not sure why...maybe the KAPT update?

[dstepanov]

I have tried and the exception is io.micronaut.core.reflect.exception.InstantiationException: Argument count [2] doesn't match required argument count: 0
Maybe the constructor order is different.
I see the same test without the exception in KSP so it should be fine

[timyates]

This label was duplicated, I removed the general one and left the more specific

[sdelamo]

ping @melix

[sdelamo]

I would like to merge this PR into 4.2.x

@melix or @graemerocher would you like to review it?

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information