Skip to content

Conversation

ArnyminerZ
Copy link
Member

@ArnyminerZ ArnyminerZ commented Jun 3, 2025

Purpose

With Android 16 scheduled for June, I think we should start the migration to SDK 36.

I've followed the official docs:

Short description

I'd say there isn't much to migrate. The only thing I've done is to forcefully enable enableOnBackInvokedCallback since it will be enabled by default on Android 16, and that would bring consistency.

I've tested it out, and it works fine specially after #1495.

The only thing that bothers me a bit is this, but I guess it's nothing new.

Checklist

  • The PR has a proper title, description and label.
  • I have self-reviewed the PR.
  • I have added documentation to complex functions and functions that can be used by other modules.
  • I have added reasonable tests or consciously decided to not add tests.

Signed-off-by: Arnau Mora <[email protected]>
@ArnyminerZ ArnyminerZ requested a review from Copilot June 3, 2025 10:40
@ArnyminerZ ArnyminerZ self-assigned this Jun 3, 2025
@ArnyminerZ ArnyminerZ added the refactoring Internal improvement of existing functions label Jun 3, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Android SDK configuration to version 36 in preparation for Android 16 compatibility. Key changes include adding the enableOnBackInvokedCallback attribute in AndroidManifest.xml and updating compileSdk and targetSdk settings in build.gradle.kts.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
app/src/main/AndroidManifest.xml Added a copyright notice and forcefully enabled on-back callback support.
app/build.gradle.kts Updated compileSdk and targetSdk from 35 to 36 to align with Android 16.

@ArnyminerZ ArnyminerZ requested a review from sunkup June 3, 2025 10:43
@ArnyminerZ ArnyminerZ marked this pull request as ready for review June 3, 2025 10:43
Copy link
Member

@rfc2822 rfc2822 left a comment

Choose a reason for hiding this comment

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

Looks good. 🤞🏻

@rfc2822 rfc2822 requested a review from sunkup June 4, 2025 12:59
@rfc2822 rfc2822 merged commit ba4d3b2 into main-ose Jun 4, 2025
8 checks passed
@rfc2822 rfc2822 deleted the sdk-36 branch June 4, 2025 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Internal improvement of existing functions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants