fix/pr 0 (#1691)

* support noop pr service to avoid PR interactions post merge

* fix noop issue

Author: motatoes

libs/spec/providers.go

@@ -177,6 +177,8 @@ type VCSProviderBasic struct{}
 
 func (v VCSProviderBasic) GetPrService(vcsSpec VcsSpec) (ci.PullRequestService, error) {
 	switch vcsSpec.VcsType {
+	case "noop":
+		return ci.MockPullRequestManager{}, nil
 	case "github":
 		token := os.Getenv("GITHUB_TOKEN")
 		if token == "" {
@@ -200,6 +202,8 @@ func (v VCSProviderBasic) GetPrService(vcsSpec VcsSpec) (ci.PullRequestService,
 
 func (v VCSProviderBasic) GetOrgService(vcsSpec VcsSpec) (ci.OrgService, error) {
 	switch vcsSpec.VcsType {
+	case "noop":
+		return ci.MockPullRequestManager{}, nil
 	case "github":
 		token := os.Getenv("GITHUB_TOKEN")
 		if token == "" {

next/services/runs.go

@@ -61,6 +61,8 @@ func RunQueuesStateMachine(queueItem *model.DiggerRunQueueItem, service ci.PullR
 			log.Printf("could not get spec: %v", err)
 			return fmt.Errorf("could not get spec: %v", err)
 		}
+		// temp: override vcs type to use "noop"
+		spec.VCS.VcsType = "noop"
 
 		vcsToken, err := GetVCSTokenFromJob(*planJob, gh)
 		if err != nil {
@@ -150,6 +152,8 @@ func RunQueuesStateMachine(queueItem *model.DiggerRunQueueItem, service ci.PullR
 			log.Printf("could not get spec: %v", err)
 			return fmt.Errorf("could not get spec: %v", err)
 		}
+		// temp: override vcs type to use "noop"
+		spec.VCS.VcsType = "noop"
 
 		vcsToken, err := GetVCSTokenFromJob(*job, gh)
 		if err != nil {

next/services/spec.go

@@ -143,7 +143,10 @@ func GetRunNameFromJob(job model.DiggerJob) (*string, error) {
 	requestedBy := jobSpec.RequestedBy
 	prNumber := *jobSpec.PullRequestNumber
 
-	runName := fmt.Sprintf("[%v] %v %v By: %v PR: %v", batchIdShort, diggerCommand, projectName, requestedBy, prNumber)
+	runName := fmt.Sprintf("[%v] %v %v By: %v", batchIdShort, diggerCommand, projectName, requestedBy)
+	if prNumber != 0 {
+		runName += fmt.Sprintf(" PR: %v", prNumber)
+	}
 	return &runName, nil
 }