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にHTMLの自動push機能を追加 #151

Merged
merged 2 commits into from
Jun 6, 2016

Conversation

tatsuya6502
Copy link
Member

  • master ブランチの CI の際に、生成した HTML を自動 push する機能を追加します。
    • master ブランチに対して、public ディレクトリの変更内容を push
    • gh-pages ブランチに対して、master の public ディレクトリの内容を push
    • ただし、現時点では、1.9 ディレクトリの内容は、実際には1.6のものなので、gh-pages へは push しないようにしています
  • gh-pages ブランチでは CI が走らないようにします。

fork したリポジトリでのテスト結果

@KeenS レビューをお願いします。あと、gmail のアドレス(コミットログに載っているアドレス)宛に、CircleCI の invitation を送りましたので、ご確認をお願いします。

- Add deployment step for master branch to push generated HTML files
  to both master and gh-pages branches.
- Disable CI on gh-pages branch
- Add help message to setup-rust.sh
@tatsuya6502 tatsuya6502 mentioned this pull request Jun 6, 2016
Closed
6 tasks
# Recreate the worktree for gh-pages
rm -rf ./gh-pages || true
git worktree prune || true
git branch -D gh-pages || true
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nits] エラーメッセージが出ているようなので 2> /dev/null するといいかもしれません。

@KeenS
Copy link
Member

KeenS commented Jun 6, 2016

ありがとうございます。これでもうレビュー以外は手が掛からなくなりましたね。

1点だけ些細な点ですがコメントしました。無視しても構いませんし、修正後にマージされても構いません。

@tatsuya6502
Copy link
Member Author

@KeenS レビューありがとうございました。git branch -D ... || true のエラーメッセージは、アドバイス通り、2> /dev/null で表示されないようにしました。

念のため、私の fork の方の master で CI して、エラーが表示されないことを確認しました(tatsuya6502 > trpl > master build 62) この内容でマージします。

@tatsuya6502 tatsuya6502 merged commit 7ef58d9 into rust-lang-ja:master Jun 6, 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