Review "failing" sample files for multiple failures, split into separate files.
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Esamina i file in snifftests/files le cui prime righe elencano più regole non superate. Verifica che ogni errore sia distinto; suddividi i file quando possibile, oppure esamina la sovrapposizione dei set di regole CakePHP e PSR-2 quando gli errori riguardano lo stesso problema. Il lavoro è completato quando gli esempi interessati presentano errori atomici o è stato modificato il set di regole.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.)
- Lingua principale
- PHP
- Stelle
- 0
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
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.
Altre issue di loadsys/loadsys_codesniffer
-
enhancement
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
loadsys/loadsys_codesniffer#51 · 1 commento ·
-
question
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
-
.editorconfig to prevent stripping trailing space, adding final newline in snifftests/files/. Apertaenhancement help-wanted
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 45/100
-
enhancement help-wanted
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
bug help-wanted
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
Tutte le issue di loadsys/loadsys_codesniffer
Issue simili
-
Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
Automattic/safe-publish#594 ·
-
needs-triage Platform(Default)
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
HttpClient
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
symfony/symfony-docs#23092 ·
-
sync-en
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
-
sync-en
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 95/100