prettier / prettier/plugin-php
false, true and null case collision with phpcs
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- PHP
- Estrellas
- 1.9k
- Forks
- 139
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Reproduce el problema con la entrada PHP proporcionada usando prettier@2.0.5 y prettier/plugin-php@0.14.3; después, inspecciona la ruta del formateador que gestiona FALSE, TRUE y NULL. Se considera terminado cuando el formateo conserva los literales en mayúsculas requeridos por los Drupal coding standards sin producir el error de phpcs indicado.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- php
- Área
- tooling
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 35/100