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

📝 formatting compat issues #914

Closed
4 tasks done
nstepien opened this issue Nov 27, 2023 · 2 comments · Fixed by #1023
Closed
4 tasks done

📝 formatting compat issues #914

nstepien opened this issue Nov 27, 2023 · 2 comments · Fixed by #1023
Labels
A-Formatter Area: formatter L-JavaScript Language: JavaScript and super languages S-Bug-confirmed Status: report has been confirmed as a valid bug S-Help-wanted Status: you're familiar with the code base and want to help the project

Comments

@nstepien
Copy link
Contributor

nstepien commented Nov 27, 2023

Environment information

Playground

What happened?

Hey, it's me again with more formatting discrepancies I found in our codebases.

Expected result

Prettier compatibility.

Code of Conduct

  • I agree to follow Biome's Code of Conduct
@faultyserver
Copy link
Contributor

The first bug here happens because the formatter uses a BestFitting set when laying out arguments to a call expression, and it somehow manages to pick the middle variant rather than falling back to the fully-broken-out layout. I've been trying to address that to solve a number of other prettier diffs (like prettier/js/arrows/curried and the like), and i've got this diff here that gets sort of close: https://github.com/biomejs/biome/compare/main..challenge/arrow-curry.

I think it's on the right track, but not at all close yet, I might spend more time looking at it, but also happy if anyone else has an intuition and can get it fixed faster lol

@nstepien nstepien changed the title 🐛 formatting compat issues 📝 formatting compat issues Nov 27, 2023
@Conaclos Conaclos added S-Help-wanted Status: you're familiar with the code base and want to help the project A-Formatter Area: formatter L-JavaScript Language: JavaScript and super languages S-Bug-confirmed Status: report has been confirmed as a valid bug labels Nov 29, 2023
@nstepien
Copy link
Contributor Author

nstepien commented Dec 3, 2023

Thank you all!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Formatter Area: formatter L-JavaScript Language: JavaScript and super languages S-Bug-confirmed Status: report has been confirmed as a valid bug S-Help-wanted Status: you're familiar with the code base and want to help the project
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants