-
Notifications
You must be signed in to change notification settings - Fork 15.1k
[ja] re: Minor update for content/ja/docs/contribute/localization.md #51357
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
[ja] re: Minor update for content/ja/docs/contribute/localization.md #51357
Conversation
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
とりあえず立てておきます。お手数をおかけして大変申し訳ありません。 この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.
ありがとうございます、数点コメントさせていただきました
/cc t-inu |
/hold |
LGTM, other than okabe-san's comments, thank you~! |
/label tide/merge-method-squash |
#51357 (comment) の確認だけお願いします、それ以外はLGTMです |
4c63e61
to
0d44ec6
Compare
t-inu Okabe-Junya nasa9084 レビューありがとうございます。反映しました。#51357 (comment) |
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.
ありがとうございます、nit commentを残しましたがLGTMです
/lgtm
LGTM label has been added. Git tree hash: 25ed2dcc748fe8dc0e17fa87d6ef55bbcfd0ac22
|
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
LGTM label has been added. Git tree hash: 6dd39e550c48457a91f58a10a6c1a0639a9d5766
|
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.
repo全体でNetlifyのbuildに失敗する問題が発生していました。upstream/mainに修正が入って解消されたので、upstream/main でこのPRをrebaseしていただきたいです
/assign
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Okabe-Junya The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com> Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>
Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>
Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com>
977153a
to
648585f
Compare
@Okabe-Junya ありがとうございます。 今更ですが、自分のレビューコメントがpendingとなっていて、未ログイン時に自分自身のコメントが表示されないことに気づきました。 レビュー頂いた際のレスポンスのうち、きちんと残した方が良いものについては、メインスレッドで議論した方が良さそうだなと思いました。 |
Pending状態のコメントは、本人以外は誰からも見えないと思います。 |
そうなのですね。 ご教示ありがとうございます。 |
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
/unhold
LGTM label has been added. Git tree hash: 5652f14427d154ccc344df0349bba53276286e7b
|
…ubernetes#51357) * [ja] modify content/ja/docs/contribute/localization.md * Apply suggestions from code review Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com> Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com> * apply suggestion from code review, fix protocol schema of links * Update content/ja/docs/contribute/localization.md Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com> * Update content/ja/docs/contribute/localization.md Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com> --------- Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com> Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>
…ubernetes#51357) * [ja] modify content/ja/docs/contribute/localization.md * Apply suggestions from code review Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com> Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com> * apply suggestion from code review, fix protocol schema of links * Update content/ja/docs/contribute/localization.md Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com> * Update content/ja/docs/contribute/localization.md Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com> --------- Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com> Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>
…ubernetes#51357) * [ja] modify content/ja/docs/contribute/localization.md * Apply suggestions from code review Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com> Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com> * apply suggestion from code review, fix protocol schema of links * Update content/ja/docs/contribute/localization.md Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com> * Update content/ja/docs/contribute/localization.md Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com> --------- Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com> Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>
…ubernetes#51357) * [ja] modify content/ja/docs/contribute/localization.md * Apply suggestions from code review Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com> Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com> * apply suggestion from code review, fix protocol schema of links * Update content/ja/docs/contribute/localization.md Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com> * Update content/ja/docs/contribute/localization.md Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com> --------- Co-authored-by: inukai <82919057+t-inu@users.noreply.github.com> Co-authored-by: Junya Okabe <86868255+Okabe-Junya@users.noreply.github.com>
Reopened PR, which is originated from #49695 .
Description
This PR contains a small modification on content/ja/docs/contribute/localization.md, which provides more clear translation workflow to support decisions by contributors.
Reviewed at once by @nasa9084 and @Okabe-Junya .
(Thanks for your kindly mention this, and sorry for bothering you!)
This PR is just related to Japanese localization, so Japanese comments or discussions are prefered/reccomended.
Issue
Closes: #49285
/area localization
/language ja