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

[WIP] master にマージされたら自動的にデプロイされるようにする #136

Closed
wants to merge 2 commits into from

Conversation

yucao24hours
Copy link
Collaborator

@yucao24hours yucao24hours commented May 3, 2017

やること

PR が master ブランチにマージされたら自動的にデプロイされるようにします。

💭 考え中のこと

このケースで「ステージング」をどう扱うかについて

今用意している「ステージング」というのは、ステージングというよりデモ環境に近い位置づけになっている気がしてきた。
今の「ステージング」にも、master にマージされたのをきっかけにして自動的にデプロイされるようにするのはできなくもないと思うけど、そうすることは意味があるんだろうか?
わたしたちが欲しいのが "「本番(master)に入れる前に動作確認したい」という用途で使う環境" なら、ステージングにも自動デプロイするようにするということはいらないかもしれない。
また、それ用のしくみがあるのでそっちを使うようにしたほうがいいかなと思った。

TODO

  • アプリ名を Travis の環境変数に入れる(社内向けアプリなのでアプリ名を公開したくないため)
  • ステージングについて話し合って決める

WIP をはずす条件

ステージングをどうするかみんなで話して結論が出たらはずします。

対応 issue

#38

@yucao24hours yucao24hours force-pushed the deploy-to-heroku-by-travisci branch from 2f343ce to ba289c0 Compare May 3, 2017 08:02
@yucao24hours yucao24hours deleted the deploy-to-heroku-by-travisci branch May 3, 2017 08:18
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.

1 participant