firefox-devtools / rfcs

RFCs for changes to DevTools
15 stars 17 forks source link

Adopt Prettier #63

Open jasonLaster opened 5 years ago

jasonLaster commented 5 years ago

It would be nice to adopt prettier across devtools and perhaps m-c globally.

Benefits

Process changes

juliandescottes commented 5 years ago

When we discussed this in the tuesday node meeting, Dan Mosedale mentioned that he and Mark Banner discussed adding prettier globally to be consistent with C++ clang. Adding prettier for devtools would be a good test case for what prettier across MC would look like.

I am totally in favor of enforcing formatting for all our JS. Whether it's prettier or clang (I think some people use clang for js formatting as well?) I don't mind. I would only ask that we don't diverge from the rest of the style used in the JS codebase in mozilla-central. But according to your last edit this is part of a global effort to add formatting everywhere in m-c, so it sounds great :)

gregtatum commented 5 years ago

Yes please. This has been a huge win for the Firefox Profiler, and simplifies the code review process. I really appreciate the clang format for the C++ side of things in Gecko.

zoepage commented 5 years ago

I love prettier and would be in favor, especially as we can lint JS and CSS here.

victorporof commented 5 years ago

I've been investigating this a considerable amount the past month or so, and have a lot of data to share here. Some conversations in this document: https://docs.google.com/document/d/1P_Wcb9VwLN8GpvyHq-_2XNcbHWTHhUfu_JYgadS-7vY/edit#heading=h.jrzw5jojmbmk

I'll pop by to chat about it :)