Merge pull request #19 from paulcpederson/git-urls

Two bug fixes

Author: ungoldman

index.js

@@ -56,13 +56,27 @@ function ghRelease (options, auth, callback) {
         process.exit(0)
       }
 
-      client.releases.createRelease(releaseOptions, function (err, res) {
+      var checkCommitForm = {
+        user: releaseOptions.owner,
+        repo: releaseOptions.repo,
+        sha: releaseOptions.target_commitish
+      }
+
+      client.repos.getCommit(checkCommitForm, function (err, res) {
         if (err) {
-          console.error(err)
+          console.log('Couldn\'t find the target commit on GitHub.')
+          console.log('Make sure you\'ve pushed everything up to ' + releaseOptions.owner + '/' + releaseOptions.repo + ' and try again')
+          console.log('Target commit: ' + releaseOptions.target_commitish)
           process.exit(1)
+        } else {
+          client.releases.createRelease(releaseOptions, function (err, res) {
+            if (err) {
+              console.error(err)
+              process.exit(1)
+            }
+            callback(null, res)
+          })
         }
-
-        callback(null, res)
       })
     })
   })

lib/get-defaults.js

@@ -5,7 +5,7 @@ var exec = require('shelljs').exec
 module.exports = function getDefaults (workPath, callback) {
   var pkg = require(path.resolve(workPath, 'package.json'))
   var commit = getTargetCommitish()
-  var repoPath = pkg.repository.url.split('github.com/')[1].split('/')
+  var repoPath = pkg.repository.url.split('github.com')[1].slice(1).split('/')
   var owner = repoPath[0]
   var repo = repoPath[1].split('.git')[0]
   var logPath = path.resolve(workPath, 'CHANGELOG.md')