prettier / prettier/plugin-php
false, true and null case collision with phpcs
Nessuno ha ancora preso questa issue.
- Lingua principale
- PHP
- Stelle
- 1.9k
- Fork
- 139
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Riproduci il problema con l’input PHP fornito usando prettier@2.0.5 e prettier/plugin-php@0.14.3, quindi esamina il percorso del formatter che gestisce FALSE, TRUE e NULL. Il lavoro è completato quando la formattazione conserva i literal in maiuscolo richiesti dai Drupal coding standards senza produrre l’errore phpcs segnalato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- php
- Ambito
- tooling
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 35/100