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 にマージされたら自動的にデプロイされるようにする #137

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

yucao24hours
Copy link
Collaborator

@yucao24hours yucao24hours commented May 3, 2017

やること

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

💭 考え中のこと

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

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

TODO

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

WIP をはずす条件

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

対応 issue

#38

@yucao24hours
Copy link
Collaborator Author

@colorbox @tkm-kj
PR の desc にあるのですが、

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

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

こちらについて、ご意見をお願いしたいです! 🙏

@colorbox
Copy link
Member

ステージングというよりデモ環境に近い

確かに、バグ修正などの確認は大体ローカル環境で行っているため、現状だとステージングはデモ環境ですね。
ステージングへの自動デプロイは不要に思えます。

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