Merge pull request #221 from figo/master

BuildJob accept map as argument rather than a slice

Author: figo

README.md

@@ -45,7 +45,7 @@ if err != nil {
   panic("Something Went Wrong")
 }
 
-queueid, err := jenkins.BuildJob(ctx, "#jobname")
+queueid, err := jenkins.BuildJob(ctx, "#jobname", nil)
 if err != nil {
   panic(err)
 }

jenkins.go

@@ -267,12 +267,8 @@ func (j *Jenkins) DeleteJob(ctx context.Context, name string) (bool, error) {
 // Invoke a job.
 // First parameter job name, second parameter is optional Build parameters.
 // Returns queue id
-func (j *Jenkins) BuildJob(ctx context.Context, name string, options ...interface{}) (int64, error) {
+func (j *Jenkins) BuildJob(ctx context.Context, name string, params map[string]string) (int64, error) {
 	job := Job{Jenkins: j, Raw: new(JobResponse), Base: "/job/" + name}
-	var params map[string]string
-	if len(options) > 0 {
-		params, _ = options[0].(map[string]string)
-	}
 	return job.InvokeSimple(ctx, params)
 }