loadsys / loadsys/loadsys_codesniffer
Review "failing" sample files for multiple failures, split into separate files.
Nessuno ha ancora preso questa issue.
- Lingua principale
- PHP
- Stelle
- 0
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
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.)
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.
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.
Valutazione
- Stack tecnologico
- php
- Ambito
- testing
- Tipo di issue
- Refactoring
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100