Upgrade to Pro — share decks privately, control downloads, hide ads and more …

hub + git

hub + git

Command line github and pr-checklit

975ff6c8166f6f32889f004a904ffa39?s=128

Oursky Limited

May 12, 2015
Tweet

Transcript

  1. hub + pr Using Github Pull-request

  2. hub • https://github.com/github/hub • git + hub = cli github

    • OS X -> brew install hub • aliased as git -> eval "$(hub alias -s)"
  3. Short hand • hub clone oursky/ourd • vs git clone

    git@github.com:oursky/ourd.git • hub remote add -p rickmak • git remote add rickmak git@github.com:rickmak/ourd.git
  4. github command • hub am https://github.com/oursky/formflow/pull/ 81 • No need

    to add remote, yay • hub fork • hub browse (open browser)
  5. 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
  6. pr-checklist Automated

  7. 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
  8. As a reminder For both sender and maintainer

  9. 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
  10. Clean log git log --merges --since="1 week" --oneline