Skip to content

build: update compileSdkVersion and targetSdk to 36#427

Merged
wax911 merged 2 commits into
developfrom
build/bump-android-sdk-to-api-36
Nov 27, 2025
Merged

build: update compileSdkVersion and targetSdk to 36#427
wax911 merged 2 commits into
developfrom
build/bump-android-sdk-to-api-36

Conversation

@wax911

@wax911 wax911 commented Nov 27, 2025

Copy link
Copy Markdown
Member

AniTrend Pull Request

Thank you for contributing! Please take a moment to review our contributing guidelines to make the process easy and effective for everyone involved.

Please open an issue before embarking on any significant pull request, especially those that add a new library or change existing tests, otherwise you risk spending a lot of time working on something that might not end up being merged into the project.

Before opening a pull request, please ensure you've done the following:

  • You have followed our contributing guidelines
  • Double checked that your branch is based on develop and targets develop (where applicable)
  • Pull request has tests (if applicable)
  • Documentation is updated (if necessary)
  • Description explains the issue/use-case resolved

Description

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Enhancement (Improves existing functionality)

IMPORTANT: By submitting a patch, you agree to allow the project owners to license your work under the terms of the Apache License 2.0.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 50 to +54
internal fun Project.configureAndroid(): Unit = baseExtension().run {
compileSdkVersion(34)
compileSdkVersion(36)
defaultConfig {
minSdk = 21
targetSdk = 34
targetSdk = 36

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Install Android 36 SDK for CI builds

Raising compileSdkVersion/targetSdk to 36 means Gradle now requires the android-36 platform, but the Android CI pipeline still runs .github/actions/android/action.yml (lines 7‑15) which only sets up JDK/Gradle and never installs Android SDK platforms, so GitHub runners on ubuntu-latest will error Failed to find target with hash string 'android-36' unless that platform happens to be preinstalled. Please add sdkmanager setup or otherwise provision API 36 in CI before this bump lands.

Useful? React with 👍 / 👎.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Auto approved automated PR

@wax911 wax911 merged commit 87a1ae7 into develop Nov 27, 2025
12 checks passed
@wax911 wax911 deleted the build/bump-android-sdk-to-api-36 branch November 27, 2025 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant