Skip to content

Fix #155 - PRのCIで生成されたHTMLが、PRの内容を反映していない #157

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

Merged
merged 1 commit into from
Jun 16, 2016

Conversation

tatsuya6502
Copy link
Member

CI で HTML が生成されない問題(#155)の対応として、make に clean ターゲットを導入します。

この PR には、以下の修正が含まれます:

  • Makefile に clean ターゲットを追加することで、CI が毎回必ず HTML ファイルを生成するよう促す
  • circlecl.yml の make VERSION=...make clean all VERSION=... に変更
  • どの make ターゲットが実行されたかひと目で分かるように、Makefile に echo メッセージを追加
  • フォント、js、css、LICENSE ファイルを、1.6/static と 1.9/static に追加。(いままでは、public/1.6 だけに直接置かれていたが、そこだと make clean で削除されてしまうので問題あり。また、public/1.9 に置き忘れていた(?) ようなので、そういうミスも防止できる)
  • circleci/setup-rust.sh のミスを修正:if [ .. == .. ]if [ .. = .. ]

@KeenS さん、レビューをお願いできないでしょうか?(特に Makefile)

- Add 'clean' target to the Makefile to ensure that the CI should always
  generate HTML files.
- Update circlecl.yml to run `make clean all ...` rather than just `make ...`
- Add some echo messages to the Makefile to help to understand which make
  target have been executed.
- Add fonts, js, css and LICENSE files to 1.6/static and 1.9/static dirs.
- Fix an error in circleci/setup-rust.sh; `if [ .. == .. ]` should read
  `if [ .. = .. ]`
@KeenS
Copy link
Member

KeenS commented Jun 16, 2016

#155 のコメント読みました。生成ファイルの存在に関らずrustbookを叩くタスクを作ってもよかったかもしれませんが変なキャッシュが残るよりcleanした方がいいですね。

フォント、js、css、LICENSE ファイルを、1.6/static と 1.9/static に追加。

おっと、失礼しました。rustbookから生成されると思っていたのですが勘違いでしたね。修正ありがとうございます。

circleci/setup-rust.sh のミスを修正

レビューの時に気付けなくて申し訳ありません。

こちらのPR、良さそうなのでマージしますね。

@KeenS KeenS merged commit 6b25f8c into rust-lang-ja:master Jun 16, 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.

2 participants