Automattic / Automattic/VIP-Coding-Standards

4.0 | Remove JS/CSS Sniffs

Offen
#442 6 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
JS/CSS
Vorherrschende Sprache
PHP
Sterne
261
Forks
44
Ø Merge
19 Min.
Gemergte PRs (30 T.)
1

Beschreibung

With JS and CSS tokenizers slated to be removed in 4.0 of PHPCS, we may have to remove our JS/CSS sniffs for compatibility: https://github.com/squizlabs/PHP_CodeSniffer/issues/2448

JS sniffs:

* JS/WindowSniff.php
* JS/DangerouslySetInnerHTMLSniff.php
* JS/HTMLExecutingFunctionsSniff.php
* JS/InnerHTMLSniff.php
* JS/StringConcatSniff.php
* JS/StrippingTagsSniff.php

CSS Sniffs:

* UserExperience/AdminBarRemovalSniff.php

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginnen Sie mit der Prüfung des verknüpften PHP_CodeSniffer-Kompatibilitätsproblems und der aufgeführten JS- und CSS-Sniff-Dateien: JS/WindowSniff.php, JS/DangerouslySetInnerHTMLSniff.php, JS/HTMLExecutingFunctionsSniff.php, JS/InnerHTMLSniff.php, JS/StringConcatSniff.php, JS/StrippingTagsSniff.php und UserExperience/AdminBarRemovalSniff.php. Bestätigen Sie, ob PHPCS 4.0 deren Entfernung erfordert; abgeschlossen bedeutet, dass die betroffenen Sniffs entsprechend dieser Kompatibilitätsentscheidung konsistent behandelt werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
css, javascript, php
Bereich
tooling
Issue-Typ
Refactoring
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
38/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.