prettier / prettier/plugin-php
[PSR-12] Method visibility
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- PHP
- Sterne
- 1.9k
- Forks
- 139
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Disclaimer: I am aware that it's not make-or-break for Prettier to adhere to PSR-12.
However, now that it is officially accepted as a standard, we should discuss (and, if necessary, reject) each current violation of PSR-12 — if only to have point to refer to in future discussions or issues.
PSR-12 Violations
-
Visibility MUST be declared on all methods.
– Section 4.4Solving this reliably would need an
--engineflag if we attempt to support PHP 4 (and, therefore, thevarkeyword).
Beitragsleitfaden
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
Beginne mit der Anforderung aus PSR-12 Abschnitt 4.4 und dem verlinkten Playground und prüfe anschließend die referenzierte --engine-Diskussion in Issue #939. Ermittle, ob die Sichtbarkeit von Methoden unterstützt werden kann, während PHP 4 und das Schlüsselwort var unterstützt werden; abgeschlossen bedeutet, dass das Projekt diesen aufgeführten Verstoß entweder implementiert oder ausdrücklich abgelehnt hat.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- tooling
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100