Review all sniffs for compatibility with PHPCS >= 4.0
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 28/100
Rechercherichtung
Es sind keine bestimmten Dateien oder Tests genannt. Beginne damit, alle VIPCS-Sniffs anhand der aufgeführten Änderungen in PHPCS 4.x zu überprüfen, und untersuche anschließend die Diskussion darüber, ob die JavaScript- und CSS-Sniffs entfernt oder beibehalten werden sollen. Als erledigt gilt die Aufgabe, wenn die Kompatibilitätsfeststellungen dokumentiert und eine Entscheidung zu den betroffenen Sniffs getroffen wurde.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Development on PHPCS 4.x has started a while back.
While there is no timeline known for when it will be ready/released, some (breaking) changes which PHPCS 4.x will contain are already known.
Notable changes as known at the time of writing this:
- The minimum supported PHP version is slated to become PHP 7.2.
- Support for the PEAR installation method will be removed.
- Support for JS and CSS sniffs will be removed.
- Support for the "old-style" ignore annotations, like
@codingStandardsIgnoreLinewill be removed. These were deprecated in PHPCS 3.2.0. - Support for the "old-style" array properties in rulesets, where array items would be passed via a comma separated value instead of as individual items. This was deprecated in PHPCS 3.3.0.
- Deprecated tokens will be removed, most notably
T_ARRAY_HINTandT_RETURN_TYPEwhich were both deprecated in PHPCS 3.3.0.
I expect most of this won't have much effect on the sniffs in VIPCS. If/when more changes become known, I will update the above list.
However, a decision is needed about the future of the sniffs which look at JS/CSS code. There is no urgency (yet), but I'm opening this issue now to raise awareness and open the discussion about this.
- Vorherrschende Sprache
- PHP
- Sterne
- 261
- Forks
- 44
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus Automattic/VIP-Coding-Standards
-
AlwaysReturnInFilter: isInsideIfConditonal() guards the conditions array after reading itEvtl. vergeben @tomjn hat das vor 7 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
-
Bug: PreGetPosts warns when the early is_main_query() return is not the first statement in its ifEvtl. vergeben @tomjn hat das vor 7 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 76/100
-
Suppress filters in get_posts false positiveEvtl. vergeben @tomjn hat das vor 8 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100
-
Breaking Change Type: Maintenance
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
Automattic/VIP-Coding-Standards#849 · 1 Kommentar ·
Alle Issues in Automattic/VIP-Coding-Standards
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
-
Made by AI module: MCP type: bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
-
maintenance
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
hawk-digital-environments/HAWKI#443 ·
Maintainer antworten meist innerhalb von 1 Tag