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.

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

fix(git): make branchName optional #27

Closed
wants to merge 1 commit into from
Closed

fix(git): make branchName optional #27

wants to merge 1 commit into from

Conversation

kkruit
Copy link
Contributor

@kkruit kkruit commented Apr 3, 2014

fixes issue #25

@caitp
Copy link
Owner

caitp commented Apr 3, 2014

Great! By the way, you don't always have to create new PRs, you can pull commits into a different branch for a different PR with git cherry-pick <SHA>, and you can remove commits from the current branch with git rebase -i HEAD~<some number> and just delete the lines with commits you don't want, and then just force-push your update.

It's nice because it means fewer issues are opened and there's less zig-zagging around. Anyways, I think we can merge this, once travis is green

@caitp caitp closed this in 4d31363 Apr 3, 2014
@kkruit
Copy link
Contributor Author

kkruit commented Apr 3, 2014

Awesome. I haven't used cherry-pick. I think it was all one commit so that would have been an issue but keeping that in mind that will come in handy. Thanks!

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.

None yet

2 participants