Automattic / Automattic/VIP-Coding-Standards

4.0 | Remove JS/CSS Sniffs

Ouverte
#442 6 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
JS/CSS
Langage dominant
PHP
Étoiles
261
Forks
44
Merge moyen
19 min
PR mergées (30 j)
1

Description

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

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par examiner le problème de compatibilité PHP_CodeSniffer lié et les fichiers de sniff JS et CSS listés : JS/WindowSniff.php, JS/DangerouslySetInnerHTMLSniff.php, JS/HTMLExecutingFunctionsSniff.php, JS/InnerHTMLSniff.php, JS/StringConcatSniff.php, JS/StrippingTagsSniff.php et UserExperience/AdminBarRemovalSniff.php. Confirmez si PHPCS 4.0 exige leur suppression ; le travail est considéré comme terminé lorsque les sniffs concernés sont traités de manière cohérente avec cette décision de compatibilité.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
css, javascript, php
Domaine
tooling
Type d'issue
Refactorisation
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
38/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.