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

Enable pending formatter features #14718

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

Conversation

Blacksmoke16
Copy link
Member

Enables the first batch of ready formatter features as called out in #13002.

NOTE: Depends on #14713

@straight-shoota straight-shoota added this to the 1.13.0 milestone Jun 16, 2024
@straight-shoota
Copy link
Member

straight-shoota commented Jun 17, 2024

I noticed an issue: #14209 and #14075 (partially) are backwards-incompatible changes. Two formatters, one with the changes enabled and one with it disabled, cannot agree on a common accepted style as a valid variant.
This can cause friction related to crystal upgrades, which should be avoidable.

We could ensure a smooth transition by first releasing a version of the formatter which doesn't change old style to new style, but accepts the new style and does not change it back to old style.

For example the change in #12951 is backwards compatible. The current formatter accepts the & style, but doesn't enforce it.

@Blacksmoke16
Copy link
Member Author

Alright, I'll see if I can put something together for that to include in this release. Then can release the part that'll migrate old to new in the next one.

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

Successfully merging this pull request may close these issues.

None yet

3 participants