Skip to content

fix build issue#45

Open
ahmedfayed29 wants to merge 2 commits intokaichii:mainfrom
ahmedfayed29:main
Open

fix build issue#45
ahmedfayed29 wants to merge 2 commits intokaichii:mainfrom
ahmedfayed29:main

Conversation

@ahmedfayed29
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

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

Updates Android SDK configuration levels to address an Android build issue by aligning the plugin library and the example app to newer API levels.

Changes:

  • Bump compileSdkVersion to 36 for the Android library module.
  • Bump compileSdkVersion and targetSdkVersion to 36 for the example Android app.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
android/build.gradle Raises library module compileSdkVersion to 36.
example/android/app/build.gradle Raises example app compileSdkVersion and targetSdkVersion to 36.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

android {
namespace "dev.kaichi.easy_pdf_viewer"
compileSdkVersion 34
compileSdkVersion 36
Copy link

Copilot AI Feb 27, 2026

Choose a reason for hiding this comment

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

compileSdkVersion was bumped to 36 while this module still uses Android Gradle Plugin 7.0.0 (see buildscript dependencies). AGP 7.0.x is not compatible with such a high compileSdkVersion and this is likely to introduce/retain build failures for consumers. Please either keep compileSdkVersion at a level supported by AGP 7.0.x, or upgrade the Android toolchain together (AGP + Gradle wrapper + related settings) and document the required SDK/AGP versions.

Suggested change
compileSdkVersion 36
compileSdkVersion 31

Copilot uses AI. Check for mistakes.
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.

2 participants