fix(gitlab): map skipped status correctly

Use gitlab.Skipped state instead of gitlab.Canceled when the
conclusion is ConclusionSkipped, so that GitLab pipelines show the
correct skipped status. Also update the skip CI path in the sinker
to use ConclusionSkipped instead of ConclusionNeutral.

Signed-off-by: Akshay Pant <akpant@redhat.com>

Author: theakshaypant

pkg/adapter/sinker.go

@@ -159,7 +159,7 @@ func (s *sinker) setupClient(ctx context.Context, repo *v1alpha1.Repository) err
 func (s *sinker) createSkipCIStatus(ctx context.Context) error {
 	statusOpts := status.StatusOpts{
 		Status:     "completed",
-		Conclusion: status.ConclusionNeutral,
+		Conclusion: status.ConclusionSkipped,
 		Title:      "CI Skipped",
 		Summary:    fmt.Sprintf("%s - CI has been skipped", s.pacInfo.ApplicationName),
 		Text:       "Commit contains a skip CI command. Use /test or /retest to manually trigger CI if needed.",

pkg/provider/gitlab/gitlab.go

@@ -308,7 +308,7 @@ func (v *Provider) CreateStatus(ctx context.Context, event *info.Event, statusOp
 
 	switch statusOpts.Conclusion {
 	case providerstatus.ConclusionSkipped:
-		state = gitlab.Canceled
+		state = gitlab.Skipped
 		statusOpts.Title = "skipped validating this commit"
 	case providerstatus.ConclusionNeutral:
 		state = gitlab.Canceled