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

@@ -158,7 +158,7 @@ func (s *sinker) setupClient(ctx context.Context, repo *v1alpha1.Repository) err
 func (s *sinker) createSkipCIStatus(ctx context.Context) error {
 	statusOpts := provider.StatusOpts{
 		Status:     "completed",
-		Conclusion: "neutral",
+		Conclusion: "skipped",
 		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

@@ -286,7 +286,7 @@ func (v *Provider) CreateStatus(ctx context.Context, event *info.Event, statusOp
 	}
 	switch statusOpts.Conclusion {
 	case "skipped":
-		statusOpts.Conclusion = "canceled"
+		statusOpts.Conclusion = "skipped"
 		statusOpts.Title = "skipped validating this commit"
 	case "neutral":
 		statusOpts.Conclusion = "canceled"