Deprecate converting issue to pull-request

This feature is likely to get dropped from GitHub API in the near future. The alternative is to simply create a new pull-request and reference the original issue in the description.
mislav · Dec 21, 2013 · 4f70dd1 · 4f70dd1 · jricketson · Jan 8, 2014
1 parent 5e130ac
commit 4f70dd1
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 19 deletions.
diff --git a/README.md b/README.md
@@ -232,9 +232,6 @@ superpowers:
     # explicit title, pull base & head:
     $ git pull-request -m "Implemented feature X" -b defunkt:master -h mislav:feature
 
-    $ git pull-request -i 123
-    [ attached pull request to issue #123 ]
-
 ### git checkout
 
     $ git checkout https://github.com/defunkt/hub/pull/73

diff --git a/features/pull_request.feature b/features/pull_request.feature
@@ -342,7 +342,11 @@ Feature: hub pull-request
       }
       """
     When I successfully run `hub pull-request -i 92`
-    Then the output should contain exactly "https://github.com/mislav/coral/pull/92\n"
+    Then the output should contain exactly:
+      """
+      https://github.com/mislav/coral/pull/92
+      Warning: Issue to pull request conversion is deprecated and might not work in the future.\n
+      """
 
   Scenario: Convert issue URL to pull request
     Given I am on the "feature" branch with upstream "origin/feature"
@@ -354,7 +358,11 @@ Feature: hub pull-request
       }
       """
     When I successfully run `hub pull-request https://github.com/mislav/coral/issues/92`
-    Then the output should contain exactly "https://github.com/mislav/coral/pull/92\n"
+    Then the output should contain exactly:
+      """
+      https://github.com/mislav/coral/pull/92
+      Warning: Issue to pull request conversion is deprecated and might not work in the future.\n
+      """
 
   Scenario: Enterprise host
     Given the "origin" remote has url "[email protected]:mislav/coral.git"

diff --git a/lib/hub/commands.rb b/lib/hub/commands.rb
@@ -115,7 +115,6 @@ def ci_status(args)
 
     # $ hub pull-request
     # $ hub pull-request "My humble contribution"
-    # $ hub pull-request -i 92
     # $ hub pull-request https://github.com/rtomayko/tilt/issues/92
     def pull_request(args)
       args.shift
@@ -173,6 +172,10 @@ def pull_request(args)
         end
       end
 
+      if options[:issue]
+        warn "Warning: Issue to pull request conversion is deprecated and might not work in the future."
+      end
+
       options[:project] = base_project
       options[:base] ||= master_branch.short_name
 

diff --git a/man/hub.1 b/man/hub.1
@@ -152,7 +152,7 @@ Opens a pull request on GitHub for the project that the "origin" remote points t
 Without \fIMESSAGE\fR or \fIFILE\fR, a text editor will open in which title and body of the pull request can be entered in the same manner as git commit message\. Pull request message can also be passed via stdin with \fB\-F \-\fR\.
 .
 .IP
-If instead of normal \fITITLE\fR an issue number is given with \fB\-i\fR, the pull request will be attached to an existing GitHub issue\. Alternatively, instead of title you can paste a full URL to an issue on GitHub\.
+Issue to pull request conversion via \fB\-i <ISSUE>\fR or \fIISSUE\-URL\fR arguments is deprecated and will likely be removed from the future versions of both hub and GitHub API\.
 .
 .TP
 \fBgit ci\-status\fR [\fB\-v\fR] [\fICOMMIT\fR]
@@ -313,9 +313,6 @@ $ git pull\-request
 
 # explicit title, pull base & head:
 $ git pull\-request \-m "Implemented feature X" \-b defunkt:master \-h mislav:feature
-
-$ git pull\-request \-i 123
-[ attached pull request to issue #123 ]
 .
 .fi
 .

diff --git a/man/hub.1.html b/man/hub.1.html
diff --git a/man/hub.1.ronn b/man/hub.1.ronn
@@ -151,9 +151,9 @@ hub also adds some custom commands that are otherwise not present in git:
     of the pull request can be entered in the same manner as git commit message.
     Pull request message can also be passed via stdin with `-F -`.
 
-    If instead of normal <TITLE> an issue number is given with `-i`, the pull
-    request will be attached to an existing GitHub issue. Alternatively, instead
-    of title you can paste a full URL to an issue on GitHub.
+    Issue to pull request conversion via `-i <ISSUE>` or <ISSUE-URL>
+    arguments is deprecated and will likely be removed from the future versions
+    of both hub and GitHub API.
 
   * `git ci-status` [`-v`] [<COMMIT>]:
     Looks up the SHA for <COMMIT> in GitHub Status API and displays the latest