-
Notifications
You must be signed in to change notification settings - Fork 72
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
Conversation
d61daf4
to
52ce460
Compare
@KentarouTakeda |
@Fendo181 |
了解です! ありがとうございます! 🙇 |
There was a problem hiding this 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 昨日の翻訳作業が自分にとって始めてのOSSにPRを出す一歩だったのですが、自分みたいな始めて翻訳する為向けにzennで手順書を書いてみました 📝 https://zenn.dev/fendo181/articles/84074e1ce63306 こちら記事は公開はしているのですが、まだSNSには告知はしてないです 🙏 内容をざっとみて、問題なければ自分のTwitterでも告知しようと思うのですがどうでしょうか...? |
@Fendo181 リポジトリ本体のREADMEにはまとめきれない細かいTipsが非常に役立ちそうです。 一点だけ、文中でリポジトリ名が |
ここ気づいてなかったです! 🙏 レビューありがとうございます! こちら18:00頃に自分のSNSアカウントで告知しようと思います! |
refs #150
php/doc-en#3901 の翻訳を担当します!
概要
該当ファイルの最新コミットがPRのマージコミットになっている事を確認した
https://github.com/php/doc-en/commits/master/reference/dom/domimplementation/createdocument.xml
翻訳文章について
tentative
の翻訳を調べると、「仮の」や「暫定的」という意味があった。同じような英文を使っている関数を参考に調べたところ、「仮の」という意味で翻訳されているのを確認したので、こちらを参考に翻訳します。-https://github.com/php/doc-ja/blob/master/reference/dom/domdocument/loadxml.xml#L77
テスト