Skip to content

Auto-update dependencies.#295

Open
dpebot wants to merge 1 commit into
masterfrom
dpebot-repositorygardener
Open

Auto-update dependencies.#295
dpebot wants to merge 1 commit into
masterfrom
dpebot-repositorygardener

Conversation

@dpebot

@dpebot dpebot commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Brought to you by your friendly Repository Gardener.

@gemini-code-assist gemini-code-assist 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.

Code Review

This pull request updates the Android SDK configuration in app/build.gradle.kts by upgrading compileSdk and targetSdk to 37, and bumps the Firebase BoM version to 34.14.1. Feedback suggests reverting the SDK versions back to 36, as API level 37 is not yet stable or widely supported, which could lead to build failures.

Comment thread app/build.gradle.kts
Comment on lines +9 to +13
compileSdk = 37
defaultConfig {
applicationId = "com.google.firebase.example.fireeats"
minSdk = 23
targetSdk = 36
targetSdk = 37

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

Upgrading compileSdk and targetSdk to 37 is premature as Android API level 37 is not yet a stable or widely released SDK version. This can cause build failures or compilation issues in environments where the SDK is not available. It is recommended to revert these to a stable or supported SDK version (such as 36 or 35).

    compileSdk = 36
    defaultConfig {
        applicationId = "com.google.firebase.example.fireeats"
        minSdk = 23
        targetSdk = 36

@dpebot dpebot force-pushed the dpebot-repositorygardener branch 2 times, most recently from bde3cfe to 0523c1e Compare June 11, 2026 09:24
@dpebot dpebot force-pushed the dpebot-repositorygardener branch from 0523c1e to 82fc518 Compare June 12, 2026 09:24
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