PHPStan - Add a rule that checks module hooks starting with `action` return void
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Start by locating the repository's existing PHPStan rule implementations and their tests, then inspect how PrestaShop module hooks are represented. Done means the rule identifies action-prefixed hooks that return content instead of void, with tests covering the expected behavior.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
To avoid unexpected behavior from action hooks, we should assert the developper does not try to return content to display as they won't be taken in account by the core.
- Lenguaje dominante
- PHP
- Estrellas
- 37
- Forks
- 20
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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.
Más de PrestaShop/php-dev-tools
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
PrestaShop/php-dev-tools#68 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
PrestaShop/php-dev-tools#58 · 1 comentario · 2 reacciones ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
PrestaShop/php-dev-tools#55 ·
-
Whitespace after file headerAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 50/100
PrestaShop/php-dev-tools#45 · 2 reacciones ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
PrestaShop/php-dev-tools#26 ·
Todos los issues de PrestaShop/php-dev-tools
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 84/100
WordPress/hosting-handbook#506 ·
Los mantenedores suelen responder en 2 días
-
Installing a plugin with composer_packages installs dev dependencies and breaks the autoloader (Error 500)Posiblemente ocupada @Boy132 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
nginx: static-file locations drop the security headers, and the dotfile deny is matched after themAbiertobug Low Security
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
backdrop-contrib/commerce#60 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 76/100