Semicolon vs colons in TypeScript types

[MichaReiser]

Description

Rome uses , to separate interface or object type literal members whereas prettier uses ;

// Prettier
type X = {
  a: string;
  b: number;
}

// Rome
type X = {
  a: string,
  b: number,
}

Proposal

Change Rome to use semicolons. Personally, I prefer commas but I adopted prettier's style and I'm fine with it.

by @cpojer

Playground

[ematipico]

This will be easy to implement, I will take care of it

[MichaReiser]

Just keep in mind that this would be a perfect first issue (part of the reason why I spent so much time writing this up ;))

[ematipico]

Sorry, I didn't see any label but it's actually true! Let's do that

[ematipico]

For whoever wants to contribute: https://github.com/rome/tools/blob/main/crates/rome_js_formatter/src/ts/lists/type_member_list.rs#L24-L31

[cpojer]

Sorry, I need this fixed. I reported it, so I fixed it :P