prettier / prettier/plugin-php

false, true and null case collision with phpcs

Abierto
#1,555 7 comentarios 0 reacciones 0 asignados Ver en GitHub

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

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.