Slide 1

Slide 1 text

hub + pr Using Github Pull-request

Slide 2

Slide 2 text

hub • https://github.com/github/hub • git + hub = cli github • OS X -> brew install hub • aliased as git -> eval "$(hub alias -s)"

Slide 3

Slide 3 text

Short hand • hub clone oursky/ourd • vs git clone [email protected]:oursky/ourd.git • hub remote add -p rickmak • git remote add rickmak [email protected]:rickmak/ourd.git

Slide 4

Slide 4 text

github command • hub am https://github.com/oursky/formflow/pull/ 81 • No need to add remote, yay • hub fork • hub browse (open browser)

Slide 5

Slide 5 text

Pull-request • hub pull-request -m "Implemented feature X" -b oursky:master -h limouren:feature • hub pull-request -F checklist.md -b oursky:master -h chpapa:wishes

Slide 6

Slide 6 text

pr-checklist Automated

Slide 7

Slide 7 text

Auto append checklist • https://github.com/rickmak/github-pr-checklist • Visit https://blooming- castle-7985.herokuapp.com/api/listenpr? repo=rickmak/waffle • cat PR.md | curl --data-binary @- -X PATCH https://blooming-castle-7985.herokuapp.com/ api/listenpr?repo=rickmak/waffle

Slide 8

Slide 8 text

As a reminder For both sender and maintainer

Slide 9

Slide 9 text

Don’t merge at web • Seriously open source project don’t use it • git inventor https://github.com/torvalds/linux/pull/17#issuecomment-5654674 • joyent/node, npm/npm, jquery(https://twitter.com/gnarf/status/378211055932936192) • “Merge pull request” Considered Harmful (http://blog.spreedly.com/2014/06/24/merge-pull- request-considered-harmful/#.VUxpy87QBo0) • You need to pull it to your environment to verify it • Give the merge a meaningful message

Slide 10

Slide 10 text

Clean log git log --merges --since="1 week" --oneline