chained method wrapping inconsistent when comments inside arguments
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 45/100
Rechercherichtung
Reproduziere die beiden Fälle aus dem verlinkten Playground und vergleiche das Wrapping von Methodenverkettungen mit und ohne den Kommentar innerhalb des Arguments von secondMethod. Verfolge, wie der Formatter Kommentare und verkettete Aufrufe behandelt, und füge dann eine Regression-Fixture mit der bereitgestellten Eingabe und den Ausgaben hinzu. Als erledigt gilt die Aufgabe, wenn der Kommentar das Wrapping nicht mehr unerwartet verändert und die Fixture erfolgreich durchläuft.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
When a comment is added inside the argument of secondMethod below, it changes the way that the chained methods are wrapped in a way that doesn't seem intuitive for it to affect.
@prettier/plugin-php v0.19.3
Playground link
Input:
<?php
$builder->firstMethod()->secondMethod([
new ClassName('c-' . $some_long_variable, [
EXTEND::THIS, LINE::SO, IT::HASNEWLINES
]),
new ClassName('c-' . $some_long_variable, [
EXTEND::THIS, LINE::SO, IT::HASNEWLINES
]),
]);
Output:
<?php
$builder
->firstMethod()
->secondMethod([
new ClassName("c-" . $some_long_variable, [
EXTEND::THIS,
LINE::SO,
IT::HASNEWLINES,
]),
new ClassName("c-" . $some_long_variable, [
EXTEND::THIS,
LINE::SO,
IT::HASNEWLINES,
]),
]);
@prettier/plugin-php v0.19.3
Playground link
Input:
<?php
$builder->firstMethod()->secondMethod([
new ClassName('c-' . $some_long_variable, [
EXTEND::THIS, LINE::SO, IT::HASNEWLINES
]),
// comment
new ClassName('c-' . $some_long_variable, [
EXTEND::THIS, LINE::SO, IT::HASNEWLINES
]),
]);
Output:
<?php
$builder->firstMethod()->secondMethod([
new ClassName("c-" . $some_long_variable, [
EXTEND::THIS,
LINE::SO,
IT::HASNEWLINES,
]),
// comment
new ClassName("c-" . $some_long_variable, [
EXTEND::THIS,
LINE::SO,
IT::HASNEWLINES,
]),
]);
- Vorherrschende Sprache
- PHP
- Sterne
- 1.9k
- Forks
- 139
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus prettier/plugin-php
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 62/100
prettier/plugin-php#2416 · 2 Kommentare · 6 Reaktionen ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 64/100
prettier/plugin-php#2498 ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 55/100
prettier/plugin-php#2495 ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100
prettier/plugin-php#2494 · 1 Kommentar ·
-
strange format issue Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100
prettier/plugin-php#2490 ·
Alle Issues in prettier/plugin-php
Ähnliche Issues
-
tooling
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
UX
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
ProfessionalWiki/NeoWiki#1525 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
OpenConext/OpenConext-engineblock#2122 ·
-
Bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
Automattic/safe-publish#594 ·