I nearly got the github flow correct on the first pass. Here's what I did:
- fork the project (via UI on github)
- clone locally
git clone git@github.com/spazm/vim-space.git
- make change to plugin.vim in my working directory, test:
vim plugin.vim
... - create an issue for the original project (via UI on github)
- commit change locally:
git add plugin.vim
git commit plugin.vim -m "[issue 7] ..." - push change to my cloned repo on github:
git push origin master
- create the patch file:
git format-patch HEAD^..HEAD
- upload the patch file to the issue. Nope, can't upload files to the issue tracker at github.
- create pull request:
git request-pull HEAD^ origin HEAD
The following changes since commit 9640d4d1ee980e352abd96e2c0ef13372d1c14cd: Merge remote branch 'remotes/origin/master' (2010-04-13 16:22:27 +0200) are available in the git repository at: git@github.com:spazm/vim-space.git master Andrew Grangaard (1): [issue 8] correct loaded_space to space_loaded in plugin plugin/space.vim | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)
- Copy and paste this pull request into issue 7.
This was close to the correct flow. While the issue tracker on github doesn't allow uploaded files in the comments, it does track pull requests. So the flow should look like:
Correct Flow
- Clone Repo (github UI)
- clone locally:
git clone git@github.com/spazm/space-vim
- Make Changes, test
vim plugin.vim
... - commit locally:
git add plugin.vim
git commit -m "loaded_space fix" - push to github:
git push origin master
- issue pull request(github UI)
- done. bask in the warm glow of improving an open source project.
The issue pull request stage will create a tracking issue. Slickness. The Clone and Pull Request are handled in the github UI. Check the Pull Requests as Github UI documentation for pretty pictures.
Proper form would be to create a topic branch in the cloned repository, this becomes more useful if you plan to make any further changes or modifications to the project. Make the upstream girl's job as easy as possible to enhance the likelihood of your changes being accepted.
No comments:
Post a Comment