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

ch08-01 リスト8-7のコードの修正 #78

Merged
merged 2 commits into from
Sep 7, 2020

Conversation

yiskw713
Copy link

@yiskw713 yiskw713 commented Sep 7, 2020

表題通り,chapter08-01のリスト8-7のコードを修正しました.

原文では

let mut v = vec![1, 2, 3, 4, 5];

let first = &v[0];

v.push(6);

println!("The first element is: {}", first);

でしたが,日本語訳のものでは

let mut v = vec![1, 2, 3, 4, 5];

let first = &v[0];

v.push(6);

と最後の行が消えておりました.
そのため,このコードの下にあるコンパイルエラーが確認できない状態になっていました.
小さな修正ではありますが,ご確認お願いいたします.

@tatsuya6502 tatsuya6502 self-requested a review September 7, 2020 11:57
@tatsuya6502 tatsuya6502 self-assigned this Sep 7, 2020
@tatsuya6502
Copy link
Member

PR(Pull Request)を提出していただき、ありがとうございます。

修正していただいたファイル docs/ch08-01-vectors.html ですが、ツールによって自動生成されるものになります。そのファイルを手で修正しても、ツールによって上書きされますので、サイト(doc.rust-jp.rs)には反映されません。

お手数ですが、生成元のMarkdownファイル src/ch08-01-vectors.md の方を修正していただけないでしょうか? docs/ch08-01-vectors.html の内容は、特に元に戻さなくても大丈夫です。

よろしくお願いいたします。

@yiskw713
Copy link
Author

yiskw713 commented Sep 7, 2020

@tatsuya6502

ご指摘ありがとうございます.仕様をあまり理解していなかったために,ご迷惑おかけしてしまい申し訳ございません.
ご指摘通り,生成元のsrc/ch08-01-vectors.mdの中身を修正させていただきました.
お手数ですがご確認よろしくお願いいたします.

@tatsuya6502
Copy link
Member

メモ

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 eb34a26 into rust-lang-ja:master-ja Sep 7, 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