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

できるだけ自然な日本語にする #33

Merged
merged 12 commits into from
Jul 3, 2020

Conversation

jnory
Copy link

@jnory jnory commented May 6, 2020

こんにちは。初めまして。
最近Rustを入門するのにこちらの本を読ませていただきました。

読むに当たって日本語としてこなれていない表現がいくつか見られましたので、
原文を参照し修正しながら読みました。

#15 を拝見しまして直接PRとの事でしたので、
修正案としてお送りさせていただきます。
ご検討いただけますと幸いです。

※ なにぶんRust素人なものですから逆に不自然にしてしまっているところなどございましたらお手数ですがご指摘ください。

どうぞよろしくお願いいたします。

@tatsuya6502 tatsuya6502 self-requested a review May 7, 2020 15:22
@tatsuya6502 tatsuya6502 self-assigned this May 7, 2020
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.

レビューが遅くなってしまい、申し訳ありません。

2点コメントしましたので、内容をご確認のうえ、修正をお願いいたします。

second-edition/src/ch04-01-what-is-ownership.md Outdated Show resolved Hide resolved
second-edition/src/ch19-04-advanced-types.md Outdated Show resolved Hide resolved
@tatsuya6502
Copy link
Member

本件ですが、レビューが遅くなってしまったこともあり、まずはレビューコメントは無視して、いまの内容のままマージしようと思います。(明日 7月4日にマージする予定です)

レビューでコメントした内容については、マージ後、当方で修正しようと思います。

@jnory
Copy link
Author

jnory commented Jul 3, 2020

こんにちは。コメントありがとうございます。お返事いただいていたのを見落としておりました。申し訳ありません。

先ほどご指摘いただいた点修正をpushいたしました。ご確認いただけますと幸いです 🙇‍♂️

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ですので、マージします。

@tatsuya6502 tatsuya6502 merged commit fcd11b3 into rust-lang-ja:second-edition-ja Jul 3, 2020
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