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

Will 510.0.0 be released? #697

Closed
kkebo opened this issue Mar 7, 2024 · 8 comments
Closed

Will 510.0.0 be released? #697

kkebo opened this issue Mar 7, 2024 · 8 comments

Comments

@kkebo
Copy link

kkebo commented Mar 7, 2024

swift-5.10.0-RELEASE has been released recently, but it lacks various features and fixes that 509.0.0 had.

For example, it doesn't include #623.

So I'm wondering if 510.0.0 will be released with more changes than swift-5.10-RELEASE.

@allevato
Copy link
Collaborator

allevato commented Mar 7, 2024

I'll be doing some cherry-picks into the 5.10 release branch before cutting a release. Expect this to happen in the next few days.

@kkebo
Copy link
Author

kkebo commented Mar 7, 2024

Thank you for the information. I'm relieved to hear that. Should I close this issue?

@allevato
Copy link
Collaborator

allevato commented Mar 7, 2024

It's fine to leave it open to track the release.

@ahoppen
Copy link
Contributor

ahoppen commented Mar 8, 2024

I just pushed 510.0.0: https://github.com/apple/swift-format/tree/510.0.0

@ahoppen ahoppen closed this as completed Mar 8, 2024
@kkebo
Copy link
Author

kkebo commented Mar 10, 2024

@ahoppen @allevato

swift-format 510.0.0 is the same commit hash as swift-5.10-RELEASE, so it's still broken.

I'll be doing some cherry-picks into the 5.10 release branch before cutting a release.

It seems that you didn't do that after all.

For example, swift-format 510.0.0 and 510.0.1 will format the following code:

let a = 0
print(consume a)

into the following one:

let a = 0
print(consumea)

swift-format 509.0.0 has no such issue because it has already been fixed in #623. swift-format 510.0.x doesn't include it. This is a regression.

@allevato
Copy link
Collaborator

So that's my fault. Older versions of swift-format had more flexibility in being able to cherry-pick late into the release cycle because it wasn't bundled with the released toolchains. Now that it is, we have to be more restrictive about what kinds of changes are cherry-picked into the release branch because they can affect the toolchain builds on swift.org. The 5.10 branch was cut fairly early, and I wasn't prepared for it.

I'm discussing with @ahoppen the best way forward; I'm hopeful that we can just cut a new branch that's separate from the official release branch, pick all the important changes into it, and then do a dot release from there.

@kkebo
Copy link
Author

kkebo commented Mar 11, 2024

Thank you for sharing the detailed information. I agree with that way of proceeding.

@ahoppen
Copy link
Contributor

ahoppen commented Mar 19, 2024

I just tagged 510.1.0, which should contain the fixes that were present in 509.0.0 but not 510.0.1

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

No branches or pull requests

3 participants