-
-
Notifications
You must be signed in to change notification settings - Fork 544
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
fix(formatter): indent instead of align in conditionals #2078
Conversation
✅ Deploy Preview for biomejs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
They still use |
Thank you, I suppose I'll have to do a check on the indentation style |
CodSpeed Performance ReportMerging #2078 will improve performances by 33.07%Falling back to comparing Summary
Benchmarks breakdown
|
1ba42c3
to
b910b27
Compare
3343383
to
7d9b9b9
Compare
Summary
Closes #1661
It turns out that prettier decided to remove the
align
inside conditionals and to use a simple indentation for nested conditions.Test Plan
Updated the snapshots
Prettier playground: https://prettier.io/playground/#N4Igxg9gdgLgprEAuEcAeAHCAnGACGATwzjwGEIAbAVwFsoAeAFT3XigBMBnPAEQCEKNegG0A5ERJiAugBo8AZVZp23PAHEEcbAEN4HClABmASwDmAPjwBePCzYI1YgEYQqcHVDEAdKN5gA-HgAglww2BD8bpQejAoWvv5IdsqqPGJQdM7aPn4w-kGh4RAAclnaDPGJ+TDJ9iqO6fAquf5tgSFhEUxslQl57XWpjXhiHHpwrTXthV0QvBN91e1JKQ6c6QBWXNBTKzWzxQBSO3H9+4N4UHAAbtoA3CCyIBAYMCbQXMigOtgRAO4ABV+CC+KB0lH+OkIX2ezl0YAA1nAYAodLQ4AAZEzXZBGCFcOBwhHI1EYHRgHFmZDhahEkCE2gmGnYOnPdAkbAmDGwCE9bBQX4mOBg-GUQnPLhUmIARWoEHgeIJ9O2aAU0rgcoVcCV4vpAEd5fBAREMGCQDouABaa5wDh2p4gcI6EyUKkUWi0HTIC2USiOqVQMwxYIwcImZzUY3abG4pBiiUgAAWMFolAA6kmTPAuOSwHAFKDsyYbtnCD6wFxYSAbnSAJKcBCosBct7BTgKIgxXWJjARQnp3QYH19kXaO6OnGE3AmnRmL09+nk7DTn2SEUtkxvR19nEwdMmDgwJPIAAcAAZntg4IaTNfZ-PvfHlc8YDpnAejyekAAmZ7UQkmHfMFaXpOBaGyDh7Q4TFPDMag5zgAAxHAvTDKkfR0KMIBAABfXCgA