Skip to content
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

[Build] Update Kotlin to 1.9.22 #77

Merged
merged 2 commits into from
Jul 5, 2024
Merged

Conversation

ParaskP7
Copy link
Contributor

@ParaskP7 ParaskP7 commented Jul 5, 2024

Required By: Dependency Analysis PR

This PR updates Kotlin to 1.9.22.

Description

This update isn't that necessary, but should be considered anyway as it will do the following:

  • Align this library, to that Kotlin version, which is already used by most of this library's clients and libraries.
  • Unblock the usage of the Dependency Analysis Gradle plugin for this project.
  • Prepares this project for the Kotlin 2+ major update.

Test

  • Smoke test the sampleapp example app.
  • Verifying that all the CI checks are successful.
  • However, if you want to be thorough about reviewing this change, you could:
    • Quickly smoke test the WCAndroid app, with this version of Media Picker, or via composite builds, and see if it works as expected.
    • Test for build time increases (local & CI).

This update isn't that necessary, but should be considered anyway as it
will do the following:
- Align this library, to that Kotlin version, which is already used by
most of this library's clients and libraries.
- Unblock the usage of the dependency analysis Gradle plugin for this
project.
- Prepares this project for the Kotlin 2+ major update.
This change is necessary because otherwise the build fails with the
below exception:

------------------------------------------------------------------------

> Task :mediapicker:kaptDebugKotlin FAILED
Annotation processors discovery from compile classpath is deprecated.
Set 'kapt.include.compile.classpath=false' to disable discovery.
Run the build with '--info' for more details.

FAILURE: Build completed with 2 failures.

1: Task failed with an exception.
-----------
* What went wrong:
Execution failed for task ':mediapicker:source-gif:kaptDebugKotlin'.
> A failure occurred while executing org.jetbrains.kotlin.gradle
 .internal.KaptWithoutKotlincTask$KaptExecutionWorkAction
   > java.lang.reflect.InvocationTargetException (no error message)

------------------------------------------------------------------------

FYI: This Dagger update is relatively safe as it is only about couple of
bug fixes:
- [2.46]: https://github.com/google/dagger/releases/tag/dagger-2.46
  - Fixed #3701 and #3838. Updated Dagger to kotlinx metadata 0.6.0.
    (7cf53031)
- [2.46.1]: https://github.com/google/dagger/releases/tag/dagger-2.46.1
  - Fixed #3701: Shade kotlinx metadata in Dagger's artifacts to prevent
    upstream dependency issues. (d593e01)
@ParaskP7 ParaskP7 merged commit 0244e25 into trunk Jul 5, 2024
13 checks passed
@ParaskP7 ParaskP7 deleted the build/update-kotlin-to-1.9.22 branch July 5, 2024 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants