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

CircleCI(プルリクエスト用) #149

Merged
merged 3 commits into from
Jun 5, 2016

Conversation

tatsuya6502
Copy link
Member

@tatsuya6502 tatsuya6502 commented Jun 4, 2016

プルリクエスト(PR)時と、masterブランチへのコミット時に、HTML を自動生成するための CI 設定を追加します。(CircleCI を使用)

  • circle.ymltools/circleci/setup-rust.sh を追加
  • Makefile がオプションとして VERSION 文字列を取るように修正。これにより、CI で 1.9 と 1.6 の両方の HTML の生成が可能になる
  • CI のステータスバッジを README.md に追加
  • CI に関する簡単な情報を CONTRIBUTING.mdREADME.md に追加
  • CI の実行が失敗しないよう public/1.9/ ディレクトリと public/1.9/index.html を追加

なお、この修正には、gh-pages への HTML の自動 push は含まれていません。

@KeenS さん、レビューをお願いします。(このプルリクにも、CI 結果が表示されるはずですので、そちらも見てください)

参考

- Add circle.yml and tools/circleci/setup-rust.sh
- Update Makefile to take an optional VERSION string so that CI can
  generate HTML for both Rust 1.9 and 1.6 docs.
- Update CONTRIBUTING.md and README.md with brief CI info.
- Add public/1.9/index.html so that initial CI run for 1.9 will not fail.
@tatsuya6502 tatsuya6502 added this to the 1.9 milestone Jun 4, 2016
@tatsuya6502 tatsuya6502 mentioned this pull request Jun 4, 2016
Closed
6 tasks
@tatsuya6502 tatsuya6502 changed the title CircleCI ベースの CI(プルリクエスト用) CircleCI(プルリクエスト用) Jun 5, 2016
@KeenS
Copy link
Member

KeenS commented Jun 5, 2016

@tatsuya6502 関連issuへのコメント及びwikiを読みました。お疲れ様です & ありがとうございます。

なお、この修正には、gh-pages への HTML の自動 push は含まれていません。

了解しました。これも早めに解決しないと手作業が残りますね

こちら、マージします。

@KeenS KeenS merged commit 77d2f5e into rust-lang-ja:master Jun 5, 2016
@tatsuya6502 tatsuya6502 deleted the circleci-initial branch June 6, 2016 02:32
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