Hi,
the new style in some cases doesn't look more readable than before, and
some files grew a lot vertically due to huge amount of added newlines.
For example:
   Assert.equal(newPrincipalURI, originalPrincipalURI,
                         "content has the same principal");
now has to be written as:
    Assert.equal(
      newPrincipalURI,
      originalPrincipalURI,
      "content has the same principal"
    );

Multi-condition ifs:
if (cond1 &&
    cond2 &&
    cond3) {
now are
if (
  cond1 &&
  cond2 &&
  cond3
) {

Also assignments
    let revHost = host.split("")
                                .reverse()
                                .join("") + ".";
now are
    let revHost =
      host
        .split("")
        .reverse()
        .join("") + ".";

Are all of these expected, or are there parts that should land yet?

Cheers,
Marco
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Reply via email to