1620556 - Upgrade Prettier to 1.19.x

Status: RESOLVED FIXED

(Developer Infrastructure :: Lint and Formatting, task)

Description

[Mark Banner (:standard8)]

Prettier 1.19.x has support for nullish coalescing and Optional chaining:

https://prettier.io/blog/2019/11/09/1.19.0.html

Once we've done the smaller update to 1.18.x (bug 1620218), we should look at doing this slightly larger update - the main difference I think is that nested array/object formatting has changed.

Comment 1

[Dan Mosedale (:dmosedale, :dmose)]

Note that upgrading prettier will require a reformat of the entire tree:

We recommend pinning an exact version of prettier in your package.json as we introduce stylistic changes in patch releases.

I suspect this is fine, but is probably something we want to take into account. CCing vporof for his thoughts...

Comment 2

[Mark Banner (:standard8)]

(In reply to Dan Mosedale (:dmose, :dmosedale) from comment #1)

Note that upgrading prettier will require a reformat of the entire tree:

As we're already on prettier, this isn't a massive reformat. So this is no more extensive than running ./mach eslint --fix, and passing the result through try.

I need to take another look, but I'm pretty sure that although there's a reasonable amount of changes, we probably don't need tree-closing for this. Maybe just landing in a quiet spot.

This is what I was referring to with "the main difference I think is that nested array/object formatting has changed.".

Comment 3

[Ed Lee :Mardak]

Looks like there's more files touched in this latest update but fewer lines affected(deleted) than the previous update from bug 1620218:

1.17.0 -> 1.18.2
 232 files changed, 419 insertions(+), 1226 deletions(-)

1.18.2 -> 1.19.1:
 357 files changed, 2630 insertions(+), 1124 deletions(-)

The previous update generally consolidated multiline strings (3 lines into 1) while this update is mostly what standard8 mentioned in comment 0 with nested expansion plus some changes to not needing parens for same-operator expressions.

Comment 4

[Ed Lee :Mardak]

Attachment: Bug 1620556 - Automatic code fixes for Prettier 1.19.1 upgrade. r?Standard8!

Comment 5

[Ed Lee :Mardak]

Attachment: Bug 1620556 - Update Prettier to 1.19.1. r?Standard8!

Depends on D66128

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/50d4048d635e
Automatic code fixes for Prettier 1.19.1 upgrade. r=Standard8,remote-protocol-reviewers,marionette-reviewers,webcompat-reviewers,perftest-reviewers,sparky,whimboo,denschub
https://hg.mozilla.org/integration/autoland/rev/4c201c81234a
Update Prettier to 1.19.1. r=Standard8

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/73dac954fb5b
comm-central automatic code fixes for Prettier 1.19.1 upgrade. rs=eslint

Comment 8

[Magnus Melin [:mkmelin]]

Not sure why pulsebot hasn't noticed this landed on mozilla-central
https://hg.mozilla.org/mozilla-central/rev/4c201c81234a16e0bd63373036690ce4dd412fb3
https://hg.mozilla.org/mozilla-central/rev/50d4048d635ee7b90600a2e041595f40f56269dc

Comment 9

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/50d4048d635e
https://hg.mozilla.org/mozilla-central/rev/4c201c81234a