Automattic / Automattic/VIP-Coding-Standards

Account for upcoming change in WP 5.7 / script tag use

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

説明

PR https://github.com/WordPress/WordPress-Coding-Standards/pull/1956 in WPCS refers to changes in WP Core which will remove the use of the `` HTML tag in PHP code.

Any existing sniff which looks at the content of inline HTML and takes the `<script>` tag into account will need to be reviewed to see if it can still detect what it is looking for with the "new" way of writing this type of code.

This PR to WP Core gives some good examples of the "new" way of writing this type of code and can be used to for code samples: https://github.com/WordPress/wordpress-develop/pull/498/files

This PR shows the new functionality which will be added to Core: https://github.com/WordPress/wordpress-develop/pull/591

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

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

調査の方向性

インライン HTML 内の script タグを検査する既存の sniffs を、出発点となるリファレンスおよびコード例としてリンク先の WPCS と WordPress Core の PRs を使ってレビューします。WP 5.7 の変更後も関連するパターンを検出できるか確認してください。影響を受ける sniffs が新しいスタイルを正しく処理できれば完了です。

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

評価

技術スタック
php
領域
tooling
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
35/100

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

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