-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update the major version of Ktor #432
Conversation
Changes in core are also required: |
Kudos, SonarCloud Quality Gate passed! |
… to 4.0.6, spock to 2.3-groovy-4.0, github workflows to Java 17 (#439)
….3.0 (#444) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
….15 (#438) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Bumps [mikepenz/action-junit-report](https://github.com/mikepenz/action-junit-report) from 3.0.2 to 3.0.3. - [Release notes](https://github.com/mikepenz/action-junit-report/releases) - [Commits](mikepenz/action-junit-report@v3.0.2...v3.0.3) --- updated-dependencies: - dependency-name: mikepenz/action-junit-report dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Sergio del Amo <[email protected]>
- groovy to 3.0.12
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure what the failure is... It could be micronaut-projects/micronaut-core#8419
kotlin-runtime/src/main/kotlin/io/micronaut/kotlin/converters/FlowConverterRegistrar.java
Outdated
Show resolved
Hide resolved
ktor/src/main/kotlin/io/micronaut/ktor/env/MicronautKtorEnvironmentConfig.kt
Outdated
Show resolved
Hide resolved
This reverts commit 949d6a8.
…nmentConfig.kt Co-authored-by: Tim Yates <[email protected]>
# Conflicts: # examples/greeting/build.gradle # gradle/libs.versions.toml
# Conflicts: # kotlin-extension-functions/src/test/kotlin/io/micronaut/kotlin/context/BeanContextExtensionsTest.kt
@wetted can you resolve the conflict? |
# Conflicts: # gradle/libs.versions.toml
The GraalVM job is failing with |
@wetted Can you try applying micronaut-projects/micronaut-project-template#309 here to the graal workflow? |
[kotlin] Update common files for branch master
Kudos, SonarCloud Quality Gate passed! |
closes #413
This also fixes a prior [seemingly] bug where
MicronautKtorEnvironmentConfig.property(path)
erroneously prefixedpath
with just a dot character ifprefix
was empty (but not null), causing the call toenv.containsProperty
to fail. @graemerocher can you double check this for me please? I'm not 100% sure I understand how the new collection viewskeys()
andtoMap()
should really look, with respect toEnvironment
, not to mention myKotlin
kung-fu is a work in progress. ;)