prettier/prettier

Idempotency / Unstable method chain

Open

#2.803 geöffnet am 13. Sept. 2017

Auf GitHub ansehen
 (16 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (51.868 Stars) (4.716 Forks)batch import
area:idempotencyarea:member chainsarea:object literalshelp wantedlang:javascript

Beschreibung

Prettier Version: 1.6.1 / master

Code

(playground1) (playground2)

Expected behavior: Running prettier twice should not change format

Actual behavior: When first playground's output is pasted into input like the second playground link, its format is changed

Contributor Guide

Tech Stack
javascripttypescriptnodejs
Domain
developer experiencetooling
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
active
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
understanding of Prettier's formatting logicJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30
Research-Richtung
The issue reports that Prettier's formatting is not idempotent for certain method chains. To reproduce, run the first playground input (https://prettier.io/playground/#...) and then format the output again, which changes. Examine Prettier's printer, likely in src/printer.js, where formatting decisions for method chains are made. Look for handling of switchMap and similar calls. Compare the AST of the two outputs to identify the difference in grouping or line breaking. The fix may involve adjusting the logic that decides when to break lines or group expressions.