prettier / prettier/plugin-php
Wrapping of static method calls
オープン
まだ誰も着手していません。
- 主要言語
- PHP
- スター
- 1.9k
- フォーク
- 139
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
The plugin is currently wrapping variable static method calls before the double-colon ::. This seems wrong to me, but at least warrants some further discussion. It doesn't appear to be mentioned in PER-CS 2.0 at all, so it's not technically wrong. We do not do this wrapping direct static method calls from a class name, only from a variable.
@prettier/plugin-php v0.22.4
Playground link
Input:
<?php
$thisIsAReallyLongVariableNameUsedToTriggerTheBehaviour::staticMethod()
->anotherMethod();
Output:
<?php
$thisIsAReallyLongVariableNameUsedToTriggerTheBehaviour
::staticMethod()
->anotherMethod();
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
リンクされたPlaygroundの再現から始め、静的メソッド呼び出し周辺の改行を担当するformatterのパスを調査します。変数を使った静的呼び出しのケースを再現し、クラス名を直接使った呼び出しと比較します。想定される折り返し動作が確立され、回帰テストでカバーされたら、issueは完了とします。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- php
- 領域
- tooling
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 38/100