ompugao / english_words_battle

MIT License
1 stars 0 forks source link

circleci integration #29

Open ompugao opened 6 years ago

ompugao commented 6 years ago

ongoing... (ref #15)

tiwanari commented 6 years ago

wktk

ompugao commented 6 years ago

run test?

ompugao commented 6 years ago

why ci doesn't run on pr...!?

ompugao commented 6 years ago

try again

ompugao commented 6 years ago

the existence of reviewer doesn't matter

tiwanari commented 6 years ago

Is this ready for review?

ompugao commented 6 years ago

sorry, no, not yet.

tiwanari commented 6 years ago

no worries:P i'm just too excited about this lol

ompugao commented 6 years ago

https://discuss.circleci.com/t/circle-not-triggering-on-pull-requests/953 https://discuss.circleci.com/t/pull-requests-not-triggering-build/1213 circleci、求めているものではないのでは…?(pull requestのtestをしてくれず、mergeされたらciが走るっぽい?)

https://qiita.com/noboru_i/items/011fd5ef1808bea3d6ba version1だとできてる人いるなぁ

ompugao commented 6 years ago

うごいた!

ompugao commented 6 years ago

https://discuss.circleci.com/t/why-circleci-2-0-does-not-support-mounting-folders/11605 circleciのdockerはvolume mountをsupport しないとな

ompugao commented 6 years ago

ほー やっとpassした🦉 squashした方がいい?

tiwanari commented 6 years ago

天才 of the year だ! squashが良いかと! あとちょっとコードのコメント整理していただけると〜🙇 makeは使えなかったんですよねー?

ompugao commented 6 years ago

makeは使えたけどcircleci用のコードがmakefileにあるのはどうなんだろうなぁと思ってやめちゃった 後々circleciのcache機構?みたいなの使うかなとも思ったし

ompugao commented 6 years ago

githubにsquash and mergeという機能があるんだ…知らなかった…

tiwanari commented 6 years ago

makeは使えたけどcircleci用のコードがmakefileにあるのはどうなんだろうなぁと思ってやめちゃっ

おー、なるほど、確かに見てみると make build くらいしか再利用できそうなところないですね笑

後々circleciのcache機構?みたいなの使うかなとも思ったし

おー、なんだそれ良さそう笑

githubにsquash and mergeという機能があるんだ…知らなかった…

ありますぞい!squashが良いかと!はそれを意図してました :+1:
Settings から default の behavior を選べますよ〜 Merge pull request から変えるといいかと 個人的には squash コミットだけできるのが history が綺麗で好きです(あと # つきでコミットしてくれて PR との対応もみやすい)

tiwanari commented 6 years ago

ミスってsingle commentで全部書いてしまった…

ompugao commented 6 years ago

https://blog.opszero.com/deploying-kubernetes-secrets-with-circleci-23c5496a1027 https://circleci.com/docs/1.0/google-cloud-platform/ https://circleci.com/docs/1.0/google-auth/

tiwanari commented 6 years ago

お、circle ci上で PRODUCTIONってプレフィクスつけて環境変数を設定するようにして、ci上でスクリプト回してsecretのファイルをアドホックに作ってあげちゃう?

ompugao commented 6 years ago

k8sわからなさすぎる… なりさんが #21 でやってくれたことをstaging用のpod?deployment?にも書いてcircleciで適用すればいいだけなんだろうけど…