Skip to content

Docker ci#358

Open
sy-tencho wants to merge 8 commits intoadd-executorfrom
docker-ci
Open

Docker ci#358
sy-tencho wants to merge 8 commits intoadd-executorfrom
docker-ci

Conversation

@sy-tencho
Copy link
Collaborator

@sy-tencho sy-tencho commented Jul 12, 2019

概要

resolve #353

before #357

詳細

masterブランチの変更があるとCI上でdocker image buildしてdocker hub に自動でpushされます。

その他

謎にオールをした勢いで書いたので、ミスの原因が発見できず無駄なコミットを増やしてしまいました。申し訳ありません。けど我ながら結構良いPRだなと思うので許してください。

@sy-tencho sy-tencho changed the base branch from master to add-executor July 12, 2019 23:50
@sy-tencho sy-tencho mentioned this pull request Jul 12, 2019
@bokusunny
Copy link
Owner

bokusunny commented Jul 13, 2019

無駄なコミットを増やしてしまいました

rebaseすれば?
https://qiita.com/takke/items/3400b55becfd72769214

@sy-tencho sy-tencho force-pushed the docker-ci branch 2 times, most recently from 7722cb8 to 6263bab Compare July 13, 2019 08:38
Fix: fix typo & try again

Fix: fix notation

Fix: remove image build from work flow

Fix: remove job

work flowでjobが正しく呼ばれているかテスト

executorを変えてテスト

workflowの設定を変えてテスト

Fix: fix workflow notation

Fix: fix workflow version

Fix: notation

Add: add orbs

Fix: ネストを修正

Fix: ネストを修正

remove unnecessay diff
sy-tencho and others added 3 commits July 14, 2019 17:22
Co-Authored-By: Taishi Murakami <liverpool1407@g.ecc.u-tokyo.ac.jp>
Co-Authored-By: Taishi Murakami <liverpool1407@g.ecc.u-tokyo.ac.jp>
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