feat: pass target url

Author: bahmutov

bin/set-commit-status.js

@@ -11,7 +11,7 @@ const args = require('minimist')(allArgs, {
     sha: 'commit',
     description: 'd'
   },
-  string: ['file', 'sha', 'state', 'label', 'description']
+  string: ['file', 'sha', 'state', 'label', 'description', 'url']
 })
 
 const api = require('..')
@@ -51,6 +51,6 @@ start
       throw new Error('missing "context" property on status object')
     }
 
-    return setCommitStatus(args.state, args.description, status)
+    return setCommitStatus(args.state, args.description, status, args.url)
   })
   .catch(onError)

package-lock.json

@@ -91,7 +91,7 @@
     "stub-spawn-once": "2.3.0"
   },
   "dependencies": {
-    "@cypress/set-commit-status": "1.2.0",
+    "@cypress/set-commit-status": "1.3.0",
     "chalk": "2.4.2",
     "check-more-types": "2.24.0",
     "debug": "4.1.1",

package.json

@@ -136,7 +136,7 @@ function getCommand (args) {
   return command
 }
 
-function setCommitStatus (state, description, status) {
+function setCommitStatus (state, description, status, targetUrl) {
   la(is.maybe.unemptyString(description), 'invalid description', description)
 
   const isValidState = is.oneOf(['error', 'pending', 'failure', 'success'])
@@ -146,15 +146,19 @@ function setCommitStatus (state, description, status) {
   console.log('setting status for commit %s', status.sha)
   console.log('context: %s', context)
   console.log('state: %s', state)
+  if (targetUrl) {
+    console.log('target url: %s', targetUrl)
+    la(is.webUrl(targetUrl), 'expected target url, got', targetUrl)
+  }
 
-  // TODO: pass targetUrl
   const options = {
     owner: status.owner,
     repo: status.repo,
     sha: status.sha,
     state,
     context,
-    description
+    description,
+    targetUrl
   }
 
   debug('setting commit status %o', options)