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] Document DOMImplementation::createDocument()の翻訳 #190

Merged
merged 2 commits into from
Nov 24, 2024

Conversation

Fendo181
Copy link
Contributor

@Fendo181 Fendo181 commented Nov 24, 2024

refs #150
php/doc-en#3901 の翻訳を担当します!

概要

該当ファイルの最新コミットがPRのマージコミットになっている事を確認した

https://github.com/php/doc-en/commits/master/reference/dom/domimplementation/createdocument.xml

image

翻訳文章について

    <row>
     <entry>8.4.0</entry>
     <entry>
      The function now has the tentative return type <type>DOMDocument</type>.
     </entry>
    </row>

tentativeの翻訳を調べると、「仮の」や「暫定的」という意味があった。同じような英文を使っている関数を参考に調べたところ、「仮の」という意味で翻訳されているのを確認したので、こちらを参考に翻訳します。

    <row>
     <entry>8.4.0</entry>
     <entry>
      この関数の仮の戻り値の型が、<type>DOMDocument</type> になりました。
     </entry>
    </row>

テスト

  • jdkfx/phpdocのプレビューでcreatedocumentの翻訳が表示されている事を確認した
変更前 変更後
image image

@Fendo181 Fendo181 marked this pull request as ready for review November 24, 2024 06:36
@Fendo181
Copy link
Contributor Author

@KentarouTakeda
すみません!
こちらバイネームで聞いてしまうのですが、レビューの依頼を出す場合ってどのようにお願すればいいでしょうか? 🙏

@KentarouTakeda
Copy link
Collaborator

@Fendo181
プルリク作成状態にしておいて頂ければ大丈夫です!
きっと誰かが見てくれますand/or 私が見てマージします!

@Fendo181
Copy link
Contributor Author

了解です! ありがとうございます! 🙇

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.

ありがとうございます!
Thank you for your first contribution🎉

1コミットだけ追加させて頂きました。
ファイル冒頭のリビジョン番号を原文と合わせておくことで、原文が更新されていた場合ツールでそれを検知する、と言ったことが行えます。もし余裕があればチェック頂けると助かります。(こちらで更新するのでも大丈夫です)

@KentarouTakeda KentarouTakeda merged commit ca3cecf into php:master Nov 24, 2024
1 check passed
@Fendo181
Copy link
Contributor Author

レビューありがとうございました! 👍

ファイル冒頭のリビジョン番号を原文と合わせておくことで、原文が更新されていた場合ツールでそれを検知する、と言ったことが行えます。もし余裕があればチェック頂けると助かります。(こちらで更新するのでも大丈夫です)

この件も了解です! 🙇
次から EN-Revisionの更新も一緒に行うようにします!

@Fendo181 Fendo181 deleted the createdocument-84 branch November 24, 2024 14:40
@Fendo181
Copy link
Contributor Author

@KentarouTakeda
昨日はレビューありがとうございました!

昨日の翻訳作業が自分にとって始めてのOSSにPRを出す一歩だったのですが、自分みたいな始めて翻訳する為向けにzennで手順書を書いてみました 📝

https://zenn.dev/fendo181/articles/84074e1ce63306

こちら記事は公開はしているのですが、まだSNSには告知はしてないです 🙏

内容をざっとみて、問題なければ自分のTwitterでも告知しようと思うのですがどうでしょうか...?
(完全な個人記事なのでレビュー依頼とかではないです!時間ある時に目を通してもらえれば幸いです!)

@KentarouTakeda
Copy link
Collaborator

@Fendo181
ありがとうございます!助かります!

リポジトリ本体のREADMEにはまとめきれない細かいTipsが非常に役立ちそうです。
問題ないのでぜひぜひ告知お願いします。

一点だけ、文中でリポジトリ名が main となっている箇所が2箇所ありました。
doc-jamaster がデフォルトブランチ名になっているので、特に設定変更せずforkすると master とコマンドを打つことになります。その点だけ修正すると良いかもしれません。

@Fendo181
Copy link
Contributor Author

@KentarouTakeda

一点だけ、文中でリポジトリ名が main となっている箇所が2箇所ありました。

ここ気づいてなかったです! 🙏 レビューありがとうございます!
mainになっている箇所はmasterに変更しました!

こちら18:00頃に自分のSNSアカウントで告知しようと思います!
確認ありがとうございました! 🚀

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