remove comments

Author: eranturgeman

commands/audit/audit.go

@@ -222,7 +222,6 @@ func ProcessResultsAndOutput(auditResults *results.SecurityCommandResults, outpu
 		return
 	}
 	// Only in case Xray's context was given (!auditCmd.IncludeVulnerabilities), and the user asked to fail the build accordingly, do so.
-	// TODO eran fix the failBuild condition here as well and fix all relevant tests accordingly (run in CI) - fixed
 	if failBuild && auditResults.HasViolationContext() && results.CheckIfFailBuild(auditResults.GetScaScansXrayResults()) {
 		err = results.NewFailBuildError()
 	}

git_test.go

@@ -1,6 +1,7 @@
 package main
 
 import (
+	"github.com/jfrog/jfrog-cli-core/v2/utils/coreutils"
 	"path/filepath"
 	"testing"
 
@@ -86,14 +87,10 @@ func TestGitAuditSimpleJson(t *testing.T) {
 	)
 }
 
-// TODO eran fix this flaky test - remove Ubuntu restriction in local
 func TestGitAuditViolationsWithIgnoreRule(t *testing.T) {
-	/*
-		if !coreutils.IsLinux() {
-			t.Skip("Skipping test. This test only runs on Linux to avoid flaky tests when running in parallel tests.")
-		}
-
-	*/
+	if !coreutils.IsLinux() {
+		t.Skip("Skipping test. This test only runs on Linux to avoid flaky tests when running in parallel tests.")
+	}
 	xrayVersion, xscVersion, testCleanUp := integration.InitGitTest(t, services.MinXrayVersionGitRepoKey)
 	defer testCleanUp()