Fixes user verfication, if not allowed then don't proceed

Shivam Mukhade · Shivam Mukhade · commit 1d39fbde0d47 · 2023-01-04T19:11:43.000+05:30
if user is not allowed to run CI then don't proceed with execution,
but PAC was going ahead after reporting status instead of aborting.
this fixes it.

Signed-off-by: Shivam Mukhade &lt;smukhade@redhat.com&gt;
diff --git a/pkg/pipelineascode/match.go b/pkg/pipelineascode/match.go
@@ -136,7 +136,7 @@ is that what you want? make sure you use -n when generating the secret, eg: echo
 			if err := p.vcx.CreateStatus(ctx, p.run.Clients.Tekton, p.event, p.run.Info.Pac, status); err != nil {
 				return repo, fmt.Errorf("failed to run create status, user is not allowed to run: %w", err)
 			}
-			return repo, nil
+			return nil, nil
 		}
 	}
 	return repo, nil

Original file line number	Diff line number	Diff line change
`@@ -136,7 +136,7 @@ is that what you want? make sure you use -n when generating the secret, eg: echo`
`136`	`136`	`if err := p.vcx.CreateStatus(ctx, p.run.Clients.Tekton, p.event, p.run.Info.Pac, status); err != nil {`
`137`	`137`	`return repo, fmt.Errorf("failed to run create status, user is not allowed to run: %w", err)`
`138`	`138`	`}`
`139`		`- return repo, nil`
	`139`	`+ return nil, nil`
`140`	`140`	`}`
`141`	`141`	`}`
`142`	`142`	`return repo, nil`