<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><div class="">Hey folks,</div><div class=""><br class=""></div><div class="">The tree has been reformatted[0] using Prettier, and moving forwards we will enforce the new JS coding style across the entire codebase. Central, inbound and autoland will reopen shortly.</div><div class=""><br class=""></div><div class=""><br class=""></div><div class=""><b class="">Rebasing</b></div><div class=""><br class=""></div><div class="">If you have any pending patches touching JS code, you’ll likely require to rebase. To make this process as seamless as possible, please run `./mach bootstrap` and say yes to everything.</div><div class=""><br class=""></div><div class="">If you run into any trouble, make sure you read the wiki[1]. The parent changeset of the reformat (b7030ce607ec) has been tagged[2] as `PRE_TREEWIDE_PRETTIER_FORMAT`. We also recommend first rebasing to this parent changeset if your patches are old.</div><div class=""><br class=""></div><div class=""><br class=""></div><div class=""><b class="">Developing</b></div><div class=""><br class=""></div><div class="">We recommend setting up your editor to format on save, or alternatively using a commit hook.</div><div class=""><br class=""></div><div class="">Check out the guides on setting up your development environment here: <a href="https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style/Formatting_JS_Code_With_Prettier_and_eslint" class="">https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style/Formatting_JS_Code_With_Prettier_and_eslint</a> </div><div class=""><br class=""></div><div class=""><br class=""></div><div class=""><b class="">Configuring</b></div><div class=""><br class=""></div><div class="">Code quality configurations for eslint are entirely determined by the options defined in .eslintrc.js. However, coding style configurations are largely predetermined within Prettier itself, and we only specify a minimal set of options in .prettierrc.</div><div class=""><br class=""></div><div class="">There may be technical reasons to get Prettier to ignore a block of code (e.g. JS parser tests). For that, use `/* eslint-disable prettier/prettier */` or `// prettier-ignore`. But these ignores are for technical reasons only, not for stylistic preference reasons. See the wiki for more info[3].</div><div class=""><br class=""></div><div class=""><br class=""></div><div class="">Victor</div><div class=""><br class=""></div>[0] <a href="https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=041abaa4aa85f5a776e2a6d6fa3e6874d007faad" class="">https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=041abaa4aa85f5a776e2a6d6fa3e6874d007faad</a> <br class="">[1] <a href="https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=8484541d66d7f30289f295c2f72c79ac660b6560" class="">https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=8484541d66d7f30289f295c2f72c79ac660b6560</a> <br class="">[2] <a href="https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style/Formatting_JS_Code_With_Prettier_and_eslint#Merging_formatted_and_unformatted_code" class="">https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style/Formatting_JS_Code_With_Prettier_and_eslint#Merging_formatted_and_unformatted_code</a> <br class="">[3] <a href="https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style/Formatting_JS_Code_With_Prettier_and_eslint#Configuration" class="">https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style/Formatting_JS_Code_With_Prettier_and_eslint#Configuration</a> </body></html>