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

What are editions #8

Merged
merged 8 commits into from
Apr 4, 2021
Merged

Conversation

ksato9700
Copy link

"What are editions?" セクションを訳してみました。先にPRした #7 を先にマージしていただくとコンフリクトなくマージできると思います。

@tatsuya6502 tatsuya6502 self-requested a review January 1, 2021 01:38
@tatsuya6502 tatsuya6502 self-assigned this Jan 1, 2021
@tatsuya6502
Copy link
Member

@ksato9700 — レビューが遅くて大変申し訳ないです。

いまの状態ですと diffが多すぎて レビューできません。お手数ですが、最新のrust-lang-ja/edition-guideのmasterブランチの内容をksato9700/edition-guideのwhat-are-editionsブランチへマージしていただけないでしょうか?

例:

$ cd ksato9700/edition-guideのwhat-are-editionsがcheckoutされているディレクトリー

## upstream/masterをマージする(rust-lang-ja/edition-guideがgit remoteに
## upstreamという名前で登録されている場合)
$ git pull upstream/master

## コンフリクトがあったら対応する

## what-are-editionsブランチへpushする
$ git push origin what-are-editions

よろしくお願いします。

@ksato9700
Copy link
Author

@tatsuya6502
承知しました。rebaseしてみます。

@ksato9700
Copy link
Author

@tatsuya6502
rebaseしました。ご確認をお願いします m(.)m

Copy link
Member

@tatsuya6502 tatsuya6502 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rebaseしていただき、ありがとうございました! 非常に助かりました。

まだレビューの途中(日本語の部分をざっと読んだだけ)ですが、いったんここでお返しします。何点かコメントしましたので、ご確認と修正をお願いします。(事前に伝えてないことばかりで申し訳ないです)

src/editions/creating-a-new-project.md Outdated Show resolved Hide resolved
src/editions/creating-a-new-project.md Outdated Show resolved Hide resolved
src/editions/index.md Outdated Show resolved Hide resolved
@ksato9700
Copy link
Author

@tatsuya6502
ご指摘の点を修正してみました。よろしくお願いいたします。

Copy link
Member

@tatsuya6502 tatsuya6502 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

レビューしました。何点かコメントしましたのでご確認ください。

[nits]がついているコメントは軽微なものですので無視しても結構です。

src/editions/index.md Outdated Show resolved Hide resolved
src/editions/index.md Outdated Show resolved Hide resolved
src/editions/index.md Outdated Show resolved Hide resolved
src/editions/index.md Outdated Show resolved Hide resolved
@ksato9700
Copy link
Author

@tatsuya6502 丁寧なレビューありがとうございます。反映させてみました。

Copy link
Member

@tatsuya6502 tatsuya6502 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます。レビューOKですのでマージします。

おつかれさまでした!

p.s.
次回以降、ついでのときで構いませんので src/SUMMARY.md に翻訳済みのページのタイトルを反映していただけないでしょうか? 生成されたサイトの左側の目次にSUMMARY.mdの内容が表示されます。

@tatsuya6502 tatsuya6502 merged commit 88d311b into rust-lang-ja:master Apr 4, 2021
@ksato9700
Copy link
Author

ありがとうございます。そして src/SUMMARY.md への反映も次回以降忘れないようにします。

@ksato9700 ksato9700 deleted the what-are-editions branch April 4, 2021 10:41
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