prettier / prettier/plugin-php
false, true and null case collision with phpcs
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 1.9k
- Forks
- 139
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
phpcs using Drupal coding standards require for FALSE, TRUE and NULL to be uppercase but prettier formats them to lowercase.
phpcs error message: TRUE, FALSE and NULL must be uppercase; expected "FALSE" but found "false"
prettier@2.0.5
prettier/plugin-php@0.14.3
Input:
$vars = [
'account' => $account,
'manage_access' => FALSE,
];
Output:
$vars = [
'account' => $account,
'manage_access' => false,
];
Expected behavior:
$vars = [
'account' => $account,
'manage_access' => FALSE,
];
Prettier output:
["INFO" - 9:51:20 AM] Detected local configuration (i.e. .prettierrc or .editorconfig), VS Code configuration will not be used
["INFO" - 9:51:20 AM] Using config file at '.../.prettierrc'
["INFO" - 9:51:20 AM] Prettier Options:
{
"filepath": "...",
"parser": "php",
"useTabs": false,
"tabWidth": 2,
"endOfLine": "lf",
"braceStyle": "1tbs",
}
["INFO" - 9:51:20 AM] Formatting completed in 238.064488ms.
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 das Problem mit der bereitgestellten PHP-Eingabe unter Verwendung von prettier@2.0.5 und prettier/plugin-php@0.14.3 und untersuche anschließend den Formatter-Pfad, der FALSE, TRUE und NULL verarbeitet. Erledigt bedeutet, dass die Formatierung die von den Drupal coding standards geforderten Großbuchstaben-Literale beibehält, ohne den gemeldeten phpcs-Fehler zu erzeugen.
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
- Klar beschrieben
- Anfängerfreundlichkeit
- 35/100