fix: pass context correctly to set-commit-status

- require that this be passed in from the parsed commit message install

Author: brian-mann

bin/set-commit-status.js

@@ -3,8 +3,6 @@
 'use strict'
 
 const debug = require('debug')('commit-message-install')
-const os = require('os')
-const path = require('path')
 
 const allArgs = process.argv.slice(2)
 const args = require('minimist')(allArgs, {
@@ -49,11 +47,10 @@ start
       return
     }
 
-    if (!args.label) {
-      const name = require(path.join(process.cwd(), 'package.json')).name
-      args.label = `${os.platform()}-${os.arch()} ${name}`
+    if (!status.context) {
+      throw new Error('missing "context" property on status object')
     }
 
-    return setCommitStatus(args.label, args.state, args.description, status)
+    return setCommitStatus(args.state, args.description, status)
   })
   .catch(onError)

src/index.js

@@ -136,14 +136,13 @@ function getCommand (args) {
   return command
 }
 
-function setCommitStatus (label, state, description, status) {
-  la(is.unemptyString(label), 'missing label to put into the context', label)
+function setCommitStatus (state, description, status) {
   la(is.maybe.unemptyString(description), 'invalid description', description)
 
   const isValidState = is.oneOf(['error', 'pending', 'failure', 'success'])
   la(isValidState(state), 'invalid commit state to set', state)
 
-  const context = label
+  const context = status.context
   console.log('setting status for commit %s', status.sha)
   console.log('context: %s', context)
   console.log('state: %s', state)