-
Notifications
You must be signed in to change notification settings - Fork 4
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
contributing.md の翻訳 #8
contributing.md の翻訳 #8
Conversation
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.
手元でtextlintのチェックできました。
「Isuse」「Pull Request」の表記は良いと思います。後でprh.ymlにも反映して自動チェックできると良いですね。
1点だけ「プルリクエスト」となっている箇所があったので修正をお願いします。
contributing.md
Outdated
|
||
We'd also love to help share your work. The [`contributed` directory](./contributed) of this repository contains community-contributed modules and code. Feel free to submit a pull request to add your projects to this directory. | ||
私たちはあなたの作品を共有するのを手伝いたいと考えています。このリポジトリの[`contributed`ディレクトリ](./contributed)には、コミュニティから寄贈されたモジュールやコードが含まれています。このディレクトリにあなたのプロジェクトを追加するために、プルリクエストを自由に送信してください。 |
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.
用語の表記ゆれあります
プルリクエスト -> Pull Request
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.
Pull Requestへ修正しました。 4efdbb6
確認ありがとうございます。指摘修正しました。
試しに、prh.ymlを書いてて気づいたのですが、translation_readme.mdでは、issueやプルリクエストの表記がありましたので、別途prh.ymlを追加して対応するPRを作成し、ドキュメント間の統一をしたいです |
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.
LGTMです!
prh.ymlのPRは既存のマークダウンの表記統一と一緒にして出すという進め方が良いと思います。
contributing.mdになります。
頻出の用語ではありませんが、IssueやPull Request周りの表現はGtiHub Docsを参照しつつ、UIに合わせるかたちで英語表記としています。
@meganetaaan さんレビューお願いします。