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

4.25. Crates and Modules (1.9) #198

Merged
merged 3 commits into from
Aug 15, 2016

Conversation

Nnwww
Copy link
Contributor

@Nnwww Nnwww commented Aug 9, 2016

en bookにpatch当てましたけど必要無かったみたいですね

@Nnwww Nnwww changed the title [WIP] 4.25. Crates and Modules (1.9) 4.25. Crates and Modules (1.9) Aug 10, 2016
@Nnwww
Copy link
Contributor Author

Nnwww commented Aug 10, 2016

@KeenS @tatsuya6502
翻訳完了しました。翻訳自体は問題なさそうですが、
rustbookのビルドに失敗してciが止まっているようです。

@tatsuya6502 tatsuya6502 added this to the 1.9 milestone Aug 11, 2016
@tatsuya6502
Copy link
Member

tatsuya6502 commented Aug 11, 2016

翻訳のアップデート、ありがとうございました。

翻訳自体は問題なさそうですが、
rustbookのビルドに失敗してciが止まっているようです。

これですが、rust-by-example-ja #38 と同じく、最近 Circle CI 環境に追加された .gitconfig が原因のようです。私が対応しますね。(今夜遅くか、今週末に、PR を上げます)

https://circleci.com/gh/rust-lang-ja/the-rust-programming-language-ja/74

$ cargo install --root $RUST_HOME --git $RUSTBOOK_GIT_URL --branch $RUSTBOOK_GIT_BRANCH || true
    Updating git repository `https://github.com/tatsuya6502/rustbook.git`
warning: spurious network error (2 tries remaining): [12/-12] Malformed URL 'ssh://[email protected]:/tatsuya6502/rustbook.git'
warning: spurious network error (1 tries remaining): [12/-12] Malformed URL 'ssh://[email protected]:/tatsuya6502/rustbook.git'
error: failed to clone into: /home/ubuntu/.cargo/git/db/rustbook-71470342abf7e491

To learn more, run the command again with --verbose.

@KeenS
Copy link
Member

KeenS commented Aug 11, 2016

@Nnwww レビューしました。翻訳自体はLGTMですが念のためCIが通ってからマージしたいと思います。

@tatsuya6502
Copy link
Member

tatsuya6502 commented Aug 12, 2016

CI の失敗を防ぐための修正を PR #200 として上げました。それが master にマージされた後、その修正を @Nnwww さんのこの up19_4_25_crates_and_modules ブランチにも(git rebase かなにかで)取り込んでもらおうと思います。そうすることで、こちらでも CI が成功するようになります。

詳細は、PR #200 が master にマージされた後、また説明します。

@KeenS
Copy link
Member

KeenS commented Aug 12, 2016

@Nnwww 上記 #200 、マージしましたのでマスターをマージお願いします。

@Nnwww Nnwww force-pushed the up19_4_25_crates_and_modules branch from a823f8f to 8ec23bb Compare August 15, 2016 11:55
@Nnwww
Copy link
Contributor Author

Nnwww commented Aug 15, 2016

@tatsuya6502 @KeenS 遅れて申し訳ありません、rebaseで取り込んで通ったようなので報告します。

@KeenS
Copy link
Member

KeenS commented Aug 15, 2016

ありがとうございます。マージします。

@KeenS KeenS merged commit f57df92 into rust-lang-ja:master Aug 15, 2016
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.

3 participants