Skip to content
This repository has been archived by the owner on Mar 3, 2022. It is now read-only.

Allow turning issues into pull-requests #47

Merged
merged 2 commits into from Apr 27, 2012
Merged

Conversation

stephencelis
Copy link
Owner

Supported by the v3 API and related to issues immensely.

@stephencelis
Copy link
Owner

Want to implement this but haven't worked out the interaction yet. Suggestions are welcome. I believe it requires a pushed, accessible branch.

@nijikokun
Copy link
Author

check out hub, it has the code to do it like so:

hub pull-request -b(base) <name:branch> -h(head) <name:branch> -i <issue #>

or

hug pull-request and it opens a vi term to allow for a title -> message then you add in your other details.

@stephencelis
Copy link
Owner

Yes, but hub requires a local repo, while ghi does not. I can always replicate the functionality, but I need to test a few things out, first.

@nijikokun
Copy link
Author

I would say just check for a local repo if not found just throw them an error saying that, or just don't even allow the function to be used if not in a local repo.

E.g., converting this ticket:

  $ git br
    master
  * pull-requests
  $ ghi e 47 -H
  Issue #47 set up to track remote branch pull-requests against master.

More explicitly:

  $ ghi e 47 -Hstephencelis:pull-requests -bmaster
@stephencelis stephencelis merged commit 6606330 into master Apr 27, 2012
@nijikokun
Copy link
Author

I have a few questions about this:

  1. It works now?
  2. Does it support using a seperate user than yourself for base and head?

such as:
--base <user/group>:<branch>
--head <user/group>:<branch>

@stephencelis
Copy link
Owner

Yes, for base. Head is always the issue repo, of course. Only supported for the edit action, for now.

@nijikokun
Copy link
Author

alright nice.

@sorin-ionescu sorin-ionescu mentioned this pull request Aug 21, 2012
@stephencelis stephencelis deleted the pull-requests branch March 2, 2014 20:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants