-
Notifications
You must be signed in to change notification settings - Fork 17
Adopt Prettier #63
Comments
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 :) |
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. |
I love prettier and would be in favor, especially as we can lint JS and CSS here. |
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 :) |
It would be nice to adopt prettier across devtools and perhaps m-c globally.
Benefits
Process changes
./mach lint
will begin picking up the regressions and reporting them on try as well.The text was updated successfully, but these errors were encountered: