loadsys / loadsys/loadsys_codesniffer
Review "failing" sample files for multiple failures, split into separate files.
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 0
- Forks
- 0
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Überprüfe die Dateien in snifftests/files, deren erste Zeilen mehrere fehlschlagende Regeln auflisten. Prüfe, ob jeder Fehler eigenständig ist; teile Dateien auf, wo möglich, oder überprüfe die Überschneidung der CakePHP- und PSR-2-Regelsätze, wenn Fehler dasselbe Problem abdecken. Abgeschlossen ist die Aufgabe, wenn die betroffenen Beispiele atomare Fehler aufweisen oder ein angepasster Regelsatz vorliegt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- testing
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100