prettier / prettier/plugin-php

[PSR-12] Method visibility

オープン
#1,180 コメント 6 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

discussion psr compat
主要言語
PHP
スター
1.9k
フォーク
139
PR マージ指標
30日以内にマージされた PR はありません

説明

Disclaimer: I am aware that it's not make-or-break for Prettier to adhere to PSR-12.

However, now that it is officially accepted as a standard, we should discuss (and, if necessary, reject) each current violation of PSR-12 — if only to have point to refer to in future discussions or issues.

PSR-12 Violations
  1. Visibility MUST be declared on all methods.
    Section 4.4

    Solving this reliably would need an --engine flag if we attempt to support PHP 4 (and, therefore, the var keyword).

Playground Link (contains all listed violations)

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

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

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

PSR-12 Section 4.4 の要件とリンク先の playground から始め、その後、issue #939 で参照されている --engine の議論を確認してください。PHP 4 と var キーワードをサポートしながらメソッドの可視性を扱えるかどうかを判断してください。完了とは、プロジェクトがこの一覧に記載された違反を実装するか、明示的に却下していることを意味します。

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

評価

技術スタック
php
領域
tooling
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

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

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