-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
[JEWEL-725] Upgrade Compose to 1.8.0-alpha02 #2927
base: master
Are you sure you want to change the base?
Conversation
5df5c4b
to
a0169b1
Compare
Do I understand correctly that we are instead waiting for non broken release of compose? Or alpha02 is already fixed? |
@jakub-senohrabek no, this version is fine. alpha03's publication is problematic and we'll skip it. The GitHub CI was interrupted for some unknown reason. |
After talking to the CMP team, it looks like we might want to wait for beta01, that should be out in a couple weeks. |
False alarm, we're ok with alpha02. The potential binary incompatibility affects other CMP targets, not desktop. |
126889c
to
f80b830
Compare
This updates both the JPS and the Gradle builds. Includes bumping Skiko to 0.8.19, and switching from the combined runtime-all dependency to the separate per-platform dependencies. This also extends the Explicit API mode to tests, too, aligning us with the JPS project, where it can't only be applied to prod code. The deprecation warnings in the buildSrc have been addressed, too. ScrollingSynchronizerTest markdown indenting has been cleaned up
f80b830
to
0326c40
Compare
Not sure why an explicit merge was needed, since there were no actual conflicts, but at least GitHub won't be complaining about it now |
This updates both the JPS and the Gradle builds.
Includes bumping Skiko to 0.8.19, and switching from the combined runtime-all dependency to the separate per-platform dependencies.
Caution
Requires #2913 to be merged first
Supersedes #2896
Closes JetBrains/jewel#725