Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PHP 8.4] Function return type changesの翻訳 #209

Merged

Conversation

siwa32
Copy link
Contributor

@siwa32 siwa32 commented Dec 12, 2024

refs #150

php/doc-en#4083 を取り込みました。

  • reference/network/functions/long2ip.xml の戻り値の説明は原文に合わせて <type>文字列</type> とタグを追加したが、原文と違ってリンクにはならない
  • 他のファイルでも同様なので、そのような仕様と解釈した

@siwa32
Copy link
Contributor Author

siwa32 commented Dec 12, 2024

すみません、ビルドでこけてしまいました。
ローカルではビルドできて表示も出来たのですが...

悪い箇所あればご指摘ください。修正致します。

@siwa32
Copy link
Contributor Author

siwa32 commented Dec 12, 2024

原因わかったかもしれないのでいったん取り下げます

@siwa32 siwa32 closed this Dec 12, 2024
@siwa32 siwa32 deleted the doc-en/4083_func-return-type-changes branch December 12, 2024 16:26
@KentarouTakeda
Copy link
Collaborator

@siwa32
プルリクエストありがとうございます!

master 側の状況がCI失敗の原因だった可能性が高いです。現時点の最新へrebase頂ければ通ると思います。

@siwa32 siwa32 restored the doc-en/4083_func-return-type-changes branch December 13, 2024 04:11
@siwa32
Copy link
Contributor Author

siwa32 commented Dec 13, 2024

forkした自分のリポジトリではビルドが成功したのでreopenします

@siwa32 siwa32 reopened this Dec 13, 2024
Copy link
Collaborator

@KentarouTakeda KentarouTakeda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます。お手数おかけしました!

reference/network/functions/long2ip.xml の戻り値の説明は原文に合わせて <type>文字列</type> とタグを追加したが、原文と違ってリンクにはならない

文字列 という名前の型が無いことが原因なので、ここは訳を調整し:

<type>string</type> として返します。

例えばこの形が正解の気がするのですが、仰る通り他でも存在するため、それらも含め別で修正を検討します。

@KentarouTakeda KentarouTakeda merged commit 7eed1da into php:master Dec 13, 2024
1 of 2 checks passed
@siwa32 siwa32 deleted the doc-en/4083_func-return-type-changes branch December 15, 2024 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants