Automattic / Automattic/VIP-Coding-Standards

4.0 | Remove JS/CSS Sniffs

オープン
#442 コメント 6 件 リアクション 0 件 担当者 0 名 GitHub で見る
JS/CSS
主要言語
PHP
スター
261
フォーク
44
平均マージ
19分
マージ済み PR(30日)
1

説明

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

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず、リンクされている PHP_CodeSniffer の互換性 issue と、一覧にある JS および CSS の sniff ファイルを確認します: JS/WindowSniff.php、JS/DangerouslySetInnerHTMLSniff.php、JS/HTMLExecutingFunctionsSniff.php、JS/InnerHTMLSniff.php、JS/StringConcatSniff.php、JS/StrippingTagsSniff.php、UserExperience/AdminBarRemovalSniff.php。PHPCS 4.0 でこれらの削除が必要かどうかを確認してください。完了とは、影響を受ける sniff がその互換性に関する判断に沿って一貫して扱われていることを意味します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
css, javascript, php
領域
tooling
issue の種類
リファクタリング
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
38/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。