0

OSSにドキュメントの修正でプルリクを送ったところ拒否されましたが、その返信の内容がわかりませんでした。以下URLとその返信内容です。

https://github.com/symfony/symfony/pull/35730

返信内容:

We never add phpdocs that do not add anything that is not already part of the method signature.

Google翻訳:

メソッドシグネチャの一部ではないものを追加しないphpdocsは追加しません

自分は何がいけなかったのでしょうか。是非ご教示願えればと思います。

3
  • 4
    三重否定になっているので、日本語にされたってわかりにくいですが、少し表現を変えると「シグニチャーに既に含まれている情報しか追加して無いようなphpdocsは追加しないよ」って、ことです。「シグニチャー(メソッド名や引数名、引数型)でわかるような情報をheader docに記述しちゃいけない」と言うheader docの原則(日本のプロジェクトだと守られていないことが多いですが)を守れ、と言う趣旨でしょう。
    – OOPer
    Commented 2020年2月16日 2:28
  • ご回答ありがとうございます。 なるほど、理解しました。
    – sou9464
    Commented 2020年2月16日 2:50
  • 2
    参考までに、例えば set() method の PHPDoc ですが、この commit で削除されています。Comit log には [DI] deprecate support for non-object services という文言があって、DI 関連(Container.php, ContainerBuilder.php, ContainerInterface.php)内の set() method の PHPDoc が削除されています。
    – user39889
    Commented 2020年2月16日 3:00

1 件の回答 1

3

三重否定になっているので、日本語にされたってわかりにくいですが、少し表現を変えると「シグニチャーに既に含まれている情報しか追加して無いようなphpdocsは追加しないよ」って、ことです。

「シグニチャー(メソッド名や引数名、引数型)でわかるような情報をheader docに記述しちゃいけない」と言うheader docの原則(日本のプロジェクトだと守られていないことが多いですが)を守れ、と言う趣旨でしょう。


この投稿は @OOPer さんのコメントなどを元に編集し、コミュニティWikiとして投稿しました。

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。