Skip to content

update: upgrading kotlin version and kotlin compiler extension version #159

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Ibnyahyah
Copy link

@Ibnyahyah Ibnyahyah commented Apr 24, 2025

User description

Story: https://app.shortcut.com/smileid/story/xxx

Summary

A few sentences/bullet points about the changes

Known Issues

Any shortcomings in your work. This may include corner cases not correctly handled or issues related
to but not within the scope of your PR. Design compromises should be discussed here if they were not
already discussed above.

Test Instructions

Concise test instructions on how to verify that your feature works as intended. This should include
changes to the development environment (if applicable) and all commands needed to run your work.

Screenshot

If applicable (e.g. UI changes), add screenshots to help explain your work.


PR Type

Enhancement


Description

  • Upgrade Kotlin and related dependencies

  • Update Gradle wrapper to version 8.10.2

  • Add VSCode Java configuration settings


Changes walkthrough 📝

Relevant files
Configuration changes
settings.json
Add VSCode Java configuration settings                                     

.vscode/settings.json

  • Added new VSCode settings file with Java configuration
  • Set Java build configuration update mode to "interactive"
  • +3/-0     
    Dependencies
    gradle-wrapper.properties
    Upgrade Gradle wrapper to version 8.10.2                                 

    android/gradle/wrapper/gradle-wrapper.properties

  • Updated Gradle distribution URL from 8.9-bin.zip to 8.10.2-all.zip
  • Upgraded to newer Gradle version with full distribution package
  • +1/-1     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @prfectionist
    Copy link

    prfectionist bot commented Apr 24, 2025

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @prfectionist
    Copy link

    prfectionist bot commented Apr 24, 2025

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @@ -0,0 +1,3 @@
    {
    "java.configuration.updateBuildConfiguration": "interactive"
    Copy link
    Member

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    remove this

    //def smileVersion = findProperty('smileVersion') ?: '10.6.1-SNAPSHOT' // Uncomment this line to use the latest snapshot version also uncomment the snapshot repository
    def smileVersion = findProperty('smileVersion') ?: '10.6.0'
    def kotlinCompilerExtension = findProperty('kotlinCompilerExtensionVersion') ?: '1.5.14'
    def kotlinCompilerExtension = findProperty('kotlinCompilerExtensionVersion') ?: '2.0.0' // Update to match Kotlin version
    Copy link
    Member

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    See this - https://developer.android.com/jetpack/androidx/releases/compose-kotlin

    this is no longer needed after kotlin 2.0

    Copy link
    Member

    @jumaallan jumaallan left a comment

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    Requested some changes

    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