Merged
Conversation
Size changesDetails📦 Next.js Bundle Analysis for react-devThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
smikitky
requested changes
Jul 15, 2023
Member
smikitky
left a comment
There was a problem hiding this comment.
すみません、個別のレビューに入る前に以下の対応をお願いできないでしょうか。
- 原文と訳文で行番号を厳密に一致させる（2か所余計な改行や重複があります）
- state, ref などは文頭でも小文字にする。props などの常に複数形を使うと決めたものを除き、refs, keys などの複数形は訳文では使わない。
- 「コンテクスト」「エフェクト」などカタカナで訳すと決まっているものを英単語のままで残さない。
よろしくお願いします。
Contributor
Author
|
単語の統一と行数の修正、その他誤字の修正を行いました。再度確認をよろしくお願いいたします。
|
Contributor
Author
|
翻訳の統一は十分注意したつもりだったんですが見落としがかなりあったようです。申し訳ありません。 |
Member
|
訳語集に載っていないものもありますし、全文目を通しているのはメンテナくらいなので気にしないで大丈夫です！ https://github.com/reactjs/ja.react.dev/pull/618/files 問題なければ、上記のページから「Add suggestion to batch」機能を使ってまとめてコミットしていただけると助かります。 |
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Contributor
Author
|
レビューの変更と、その他自分で見つけた場所を修正しました。 再度確認をお願い致します。 |
smikitky
approved these changes
Jul 19, 2023
Member
smikitky
left a comment
There was a problem hiding this comment.
ありがとうございました！
2 名のレビュアーが approve してからマージされますので、このままお待ちください。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
src/content/reference/react/index.mdを翻訳しました。Issue: #452 (comment)
各ページへのリンクについて
各ページへのリンクの概要の文章は私が翻訳しましたが、各ページのリンク先の翻訳を引用したほうが良いでしょうか？
Textlintについて
#616 での通り、
textlintディレクトリに入った状態でのみtextlintが正常に動作しました。Gitへのコミット時にはtextlintを無効化してコミットしていますが、textlintでエラーや警告が出ないことは確認済みです。