☂️ Formatter 0.10.0

[MichaReiser]

Description

Goals

• Stable comments formatting 📎 Comments Formatting #2664
• Improve Prettier compatibility

Tasks

Comments (~2 weeks)

• Pre-process AST and extract comments
• Improve comments placement

Prettier Compatibility (~1 week)

High Priority

• Call Chain / Method chain feat(rome_js_formatter): Format Member Chain #3273
• Call arguments feat(rome_js_formatter): Call arguments formatting #3290
• first argument expansion feat(rome_js_formatter): Call arguments formatting #3290
• last argument expansion feat(rome_js_formatter): Call arguments formatting #3290

Bugs

• JSX Spaces 🐛 In JSX, some spaces are removed #3211 and playground

Best Effort

• Formatter: TS Intersection & Union types #3162
• Export: Respect empty lines

[denbezrukov]

@MichaReiser Hi!
I can try to help you with TS Intersection.

https://github.com/prettier/prettier/blob/68bd87b24ba00049ac74870de75596639fd967bb/src/language-js/print/type-annotation.js#L94-L120

[MichaReiser]

@MichaReiser Hi! I can try to help you with TS Intersection.

prettier/prettier@68bd87b/src/language-js/print/type-annotation.js#L94-L120

Whoops sorry. I missed this message. That would be awesome! Let me create an issue and assign that to you

Edit: I always forget that people have to comment first for me to be able to assign them o an issue. Would you mind commenting here: #3162

[nstepien]

Really hoping for a trailing commas option, it's the biggest difference with Prettier for us. 🙏
https://prettier.io/docs/en/options.html#trailing-commas

[cpojer]

Here are a few more inconsistencies: #3178.

[MichaReiser]

If anyone is interested having a look at the JSX spacing issues, let me know.

[MichaReiser]

Really hoping for a trailing commas option, it's the biggest difference with Prettier for us. pray prettier.io/docs/en/options.html#trailing-commas

We're considering adding the trailing-comma es5 option which is the default in Prettier today. Would this be sufficient for your use case or are you using none?

[nstepien]

We use "trailingComma": "none"