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

[JEWEL-725] Upgrade Compose to 1.8.0-alpha02 #2927

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rock3r
Copy link
Collaborator

@rock3r rock3r commented Jan 22, 2025

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

@hamen hamen force-pushed the sebp/jewel-725_upgrade-compose branch from 5df5c4b to a0169b1 Compare February 4, 2025 16:31
@jakub-senohrabek
Copy link
Collaborator

jakub-senohrabek commented Feb 7, 2025

Do I understand correctly that we are instead waiting for non broken release of compose? Or alpha02 is already fixed?

@rock3r
Copy link
Collaborator Author

rock3r commented Feb 7, 2025

@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.

@rock3r
Copy link
Collaborator Author

rock3r commented Feb 11, 2025

I just checked why this is red — it's only because the CI was interrupted. It will still be red if you re-run it, because master is red. The fixes are in #2941 and #2915, I won't go and fix them again in this branch, too.

@rock3r rock3r added the jewel label Feb 11, 2025
@rock3r
Copy link
Collaborator Author

rock3r commented Feb 11, 2025

After talking to the CMP team, it looks like we might want to wait for beta01, that should be out in a couple weeks.

@rock3r rock3r marked this pull request as draft February 11, 2025 17:59
@rock3r
Copy link
Collaborator Author

rock3r commented Feb 12, 2025

False alarm, we're ok with alpha02. The potential binary incompatibility affects other CMP targets, not desktop.

@rock3r rock3r marked this pull request as ready for review February 12, 2025 08:55
@rock3r rock3r force-pushed the sebp/jewel-725_upgrade-compose branch 2 times, most recently from 126889c to f80b830 Compare February 13, 2025 11:23
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
@rock3r rock3r force-pushed the sebp/jewel-725_upgrade-compose branch from f80b830 to 0326c40 Compare February 13, 2025 11:34
@rock3r
Copy link
Collaborator Author

rock3r commented Feb 13, 2025

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Compose to 1.8.0
3 participants