prettier / prettier/plugin-php
Format multiple-catch
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 1.9k
- Forks
- 139
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
We're currently not respecting line length limitations in multiple-catch blocks, which we could.
@prettier/plugin-php v0.14.0
Playground link
Input:
<?php
try {
do_something();
} catch (\Some\Rather\Obscure\Exception | \Another\Obscure\Exception $exception) {
handle_exception();
}
Current Output:
<?php
try {
do_something();
} catch (\Some\Rather\Obscure\Exception | \Another\Obscure\Exception $exception) {
handle_exception();
}
Improved Output:
<?php
try {
do_something();
} catch (
\Some\Rather\Obscure\Exception |
\Another\Obscure\Exception $exception
) {
handle_exception();
}
To be honest, the improved output is not perfect (the $exception variable is a little bit "obscured" next to the exception class), but it's better than the current behavior IMO.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit der bereitgestellten Playground-Reproduktion und durchsuche den PHP printer nach der Formatierung von multiple-catch; in der Issue wird keine Datei und kein Test genannt. Als abgeschlossen gilt die Aufgabe, wenn die Ausnahmeliste unter Einhaltung der konfigurierten Zeilenlänge umgebrochen wird, wobei die gezeigte catch-Block-Struktur erhalten bleibt und die relevanten Formatter-Tests erfolgreich sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 48/100