prettier / prettier/plugin-php
Shebang results in two newlines at the end of file
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 1.9k
- Forks
- 139
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
@prettier/plugin-php v0.22.4
Playground link
Input:
#! /usr/bin/env php
<?php
$status = 0; exit($status);
Output:
#! /usr/bin/env php
<?php
$status = 0;
exit($status);
The Markdown render is not showing the newlines, so check the Playground.
Gut instinct is that it might be triggering the code path for inline code, but not sure, so letting you tag it if it is (not that I can even add labels). I'm mulling over how minor or petty this is, but I think the inconsistency between files with & without a shebang and how no other formatter for Prettier has a double newline at the end of a file is enough of a reason.
Feel free to shoot this down if you have a reason to do so. If for nothing else, someone might stumble across this issue and not feel like they are going crazy as to why they get double newlines at the end of the file. If you want someone to blame, it's Laravel's artisan stub they put into your repo that got me going.
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
Reproduziere die Ausgabe im verlinkten Playground mit @prettier/plugin-php v0.22.4, indem du das Shebang-Beispiel mit einer äquivalenten Datei ohne Shebang vergleichst. Verfolge den Formatter-Pfad, der das Shebang verarbeitet, und überprüfe, dass die formatierte Datei mit genau einem abschließenden Zeilenumbruch statt mit zwei endet.
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
- 45/100