Skip to content
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

Bump to Compose alpha September release for 1.7.x track #2379

Merged
merged 2 commits into from
Sep 10, 2024

Conversation

oas004
Copy link
Contributor

@oas004 oas004 commented Sep 9, 2024

WHAT

Bump to Compose alpha track for 1.7.x releases

WHY

resolves #2374

HOW

Bumping the version, running the sample apps and the tasks in contribution.md

Checklist 📋

  • Add explicit visibility modifier and explicit return types for public declarations
  • Run spotless check
  • Run tests
  • Update metalava's signature text files
@@ -138,7 +138,7 @@ coil-test = { module = "io.coil-kt:coil-test", version.ref = "io-coil-kt" }
com-squareup-okhttp3-logging-interceptor = { module = "com.squareup.okhttp3:logging-interceptor", version.ref = "com-squareup-okhttp3" }
com-squareup-okhttp3-okhttp = { module = "com.squareup.okhttp3:okhttp", version.ref = "com-squareup-okhttp3" }
com-squareup-okhttp3-mockwebserver = { module = "com.squareup.okhttp3:mockwebserver", version.ref = "com-squareup-okhttp3" }
compose-bom = "androidx.compose:compose-bom:2024.09.00"
compose-bom = "androidx.compose:compose-bom:2024.09.00-alpha"
compose-animation-animationgraphics = { module = "androidx.compose.animation:animation-graphics", version.ref = "compose" }
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we strip versions from some of the dependencies? This can just come from the BOM now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah okay! Yeah I can fix that :) Thanks!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I tired to move all compose related dependencies to BOM now, but it required us using the BOM in more modules. Can revert that if there are specific reasons not to use BOM for these modules :)

Comment on lines 16 to +17
androidxCore = "1.13.1"
androidxComposeBom = "2024.09.00-alpha"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

re-order perhaps?

@yschimke yschimke merged commit 4f330f4 into google:main Sep 10, 2024
6 checks passed
@oas004 oas004 mentioned this pull request Sep 11, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants