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.

Sign up for GitHub

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

Jump to bottom

Add CoderDojo 倉敷 #337

Merged
merged 2 commits into from
Jun 1, 2018
Merged

Add CoderDojo 倉敷 #337

merged 2 commits into from
Jun 1, 2018

Conversation

naopontan
Copy link
Contributor

@naopontan naopontan commented May 31, 2018 •
edited by yasulab
Loading

cf. #335

TODO

  • 倉敷の情報を追加する
  • 倉敷の connpass id を総計情報に追加する
  • コミットを squash でまとめる

yasulab
yasulab reviewed May 31, 2018
View reviewed changes
db/dojos.yaml
created_at: '2018-05-29'
name: 倉敷 (岡山)
prefecture_id: 33
logo: "/img/dojos/kurashiki.jpg" # TODO: 画像サイズを調整する必要あり
Copy link
Member

Choose a reason for hiding this comment

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

Nice note! 📝✨

@yasulab
Copy link
Member

yasulab commented May 31, 2018

@naopontan Conflict してるので、Conflict を解消して PR がマージできる状態になってから Reviewer に Assign してください 🙏 💦

@yasulab yasulab changed the title Add kurashiki dojo #335 [WIP] Add kurashiki dojo #335 May 31, 2018
@yasulab
Copy link
Member

yasulab commented May 31, 2018

と思ったらこれは自動アサインかな...? 失礼しました 🙇 💦

@naopontan
Copy link
Contributor Author

コンフリクトの件、
最新の master ブランチを取り込む形で対応しました。
これであってるかしら...

@naopontan naopontan changed the title [WIP] Add kurashiki dojo #335 Add kurashiki dojo #335 May 31, 2018
@naopontan
Copy link
Contributor Author

@yasulab
こちら、作業としては完了しています。
タイトルに付いていた WIP を外しました（普通に github 上で編集すればいいんですね）
確認お願いします。

@yasulab
Copy link
Member

yasulab commented May 31, 2018

@naopontan コミットを Squash しましょう! 👍

WIPでPRを出して後でsquashしよう - YassLab 株式会社
https://yasslab.qiita.com/AnaTofuZ/items/0598085ca32520f145dc

@naopontan
Copy link
Contributor Author

naopontan commented Jun 1, 2018 •
edited by yasulab
Loading

@yasulab
すみません、もしよろしければ教えてください。

git rebase -i を使って ce097a0 と 7a0c856 を1つにしました。
以下は squash する前の状態です(3つのコミットのうち、下の2つです)

$ git log --date=iso  --author=naopontan
commit f5866ea8428e73320b02fe8542ed62a28c0a68b2 (HEAD -> add-kurashiki-dojo, origin/add-kurashiki-dojo, hoge)
Merge: ce097a0 394166c
Author: nao taka <[email protected]>
Date:   2018-05-31 13:53:11 +0900

    Merge branch 'master' into add-kurashiki-dojo

commit ce097a0a70a60dc8c7ed36f1146c21e1b8f55549
Author: nao taka <[email protected]>
Date:   2018-05-30 18:49:26 +0900

    :new: Add CoderDojo 倉敷 close #335

commit 7a0c856ac8ffc8dd5c973a5543eedaee55c8b773
Author: nao taka <[email protected]>
Date:   2018-05-30 17:01:56 +0900

    [WIP] :new: Add CoderDojo 倉敷 #335

    画像サイズが縦長なので、調整する必要あり。
    縦長の画像を正方形に収めるには左右に余白ができるけど、余白は白色 or 透過になると思われる。

この後、 git push --force する予定ですが、これで大丈夫でしょうか？
(纏めたい2つのコミットの間や前後には他のコミットなどもあり、壊れて取り返しがつかなくならないか不安)
force オプションを使うのになんか抵抗があって、、、

ちなみに、 rebase 後の状態は以下の通りです

$ git log --date=iso  --author=naopontan
commit 2fc0a7c0fb05c7b2874e84b3b281c4623a94ea7e (HEAD -> add-kurashiki-dojo)
Author: nao taka <[email protected]>
Date:   2018-05-30 17:01:56 +0900

    :new: Add CoderDojo 倉敷 close #335

commit 15d3b049fd96f2c5c3eee405fea400c58d8e5c57
Author: nao taka <[email protected]>
Date:   2018-05-18 11:05:22 +0900

    CoderDojo静岡の URLを変更


$ git status
On branch add-kurashiki-dojo
Your branch and 'origin/add-kurashiki-dojo' have diverged,
and have 1 and 12 different commits each, respectively.
  (use "git pull" to merge the remote branch into yours)

nothing to commit, working tree clean

@yasulab
Copy link
Member

yasulab commented Jun 1, 2018

@naopontan 今日の朝会には参加できることになったので、朝会の時に説明しますね ;)

@naopontan
Copy link
Contributor Author

はい！

@yasulab yasulab changed the title Add kurashiki dojo #335 [WIP] Add kurashiki dojo #335 Jun 1, 2018
@yasulab yasulab changed the title [WIP] Add kurashiki dojo #335 Add kurashiki dojo #335 Jun 1, 2018
@yasulab yasulab changed the title Add kurashiki dojo #335 Add CoderDojo 倉敷 dojo Jun 1, 2018
@yasulab yasulab changed the title Add CoderDojo 倉敷 dojo Add CoderDojo 倉敷 Jun 1, 2018
@naopontan naopontan force-pushed the add-kurashiki-dojo branch from f5866ea to 2fc0a7c Compare June 1, 2018 02:03
@yasulab yasulab merged commit 164f617 into master Jun 1, 2018
@yasulab yasulab deleted the add-kurashiki-dojo branch June 1, 2018 02:10
@yasulab
Copy link
Member

yasulab commented Jun 1, 2018

マージしました! ✅ PR ありがとうございます! 😸

yasulab added a commit that referenced this pull request Jun 1, 2018
@yasulab
Compress and edit CoderDojo 倉敷's logo #337
875f9c3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yasulab yasulab yasulab left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@naopontan @yasulab