Review "failing" sample files for multiple failures, split into separate files.
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Revisa los archivos de snifftests/files cuyas primeras líneas enumeran varias reglas que fallan. Comprueba si cada fallo es distinto; divide los archivos cuando sea posible, o revisa el solapamiento de los conjuntos de reglas CakePHP y PSR-2 cuando los fallos cubran el mismo problema. Se considera terminado cuando las muestras afectadas tengan fallos atómicos o se haya ajustado el conjunto de reglas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Some of the sample files currently fail multiple sniffs. Our testing harness can catch multiple sniff failures per file, but it makes the tests relatively inflexible since the files aren't "atomic" compared to the sniff rules.
It's also possible for the test files to be failing tests they really shouldn't be, such as overlapping rules that cover the same "problem" with the code. In these cases, the ruleset needs to be modified to try to only include one relevant rule to cover the failure.
Bottom line is: The files in snifftests/files that contain first lines like this:
<?php //~Standard.Section.Sniff.Rule, Another.Failing.Rule.Name
(The ones that define multiple failing rules.) Those need to be reviewed and split up where possible, or the ruleset modified to not cause overlapping failures. (The CakePHP sniffs are responsible for a lot of overlap against PSR-2, and typically the Cake sniffs lack the "autofix" capability the PSR-2 sniffs include.)
- Lenguaje dominante
- PHP
- Estrellas
- 0
- Forks
- 0
- 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 loadsys/loadsys_codesniffer
-
enhancement
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
loadsys/loadsys_codesniffer#51 · 1 comentario ·
-
question
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
-
.editorconfig to prevent stripping trailing space, adding final newline in snifftests/files/.Abiertoenhancement help-wanted
Dificultad 1/5 Menos de una hora Aptitud para principiantes 45/100
-
enhancement help-wanted
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
-
bug help-wanted
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
Todos los issues de loadsys/loadsys_codesniffer
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
codeigniter4/CodeIgniter4#10616 ·
Los mantenedores suelen responder en 1 día
-
bug code quality
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
SemanticMediaWiki/SemanticMediaWiki#7149 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
google/site-kit-wp#13825 ·
Los mantenedores suelen responder en 3 días
-
Made by AI module: MCP type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día