🐛 QD-13112 fix ignore of ignoreReturnCode option in git operations

Author: AndreiIurko

gitlab/dist/qodana-gitlab.js

@@ -51859,7 +51859,7 @@ var require_utils5 = __commonJS({
     function gitOutput(args_1) {
       return __awaiter2(this, arguments, void 0, function* (args, ignoreReturnCode = false) {
         const result = yield execAsync("git", args, true);
-        if (result.returnCode !== 0 && !ignoreReturnCode) {
+        if (result.returnCode !== 0 && ignoreReturnCode) {
           console.warn(`Git command failed: git ${args.join(" ")}
 Exit code: ${result.returnCode}
 Stdout: ${result.stdout}

gitlab/src/utils.ts

@@ -124,7 +124,7 @@ async function gitOutput(
   ignoreReturnCode = false
 ): Promise<CommandOutput> {
   const result = await execAsync('git', args, true)
-  if (result.returnCode !== 0 && !ignoreReturnCode) {
+  if (result.returnCode !== 0 && ignoreReturnCode) {
     console.warn(
       `Git command failed: git ${args.join(' ')}\nExit code: ${result.returnCode}\nStdout: ${result.stdout}\nStderr: ${result.stderr}`
     )

scan/dist/index.js

@@ -135129,6 +135129,7 @@ var require_utils10 = __commonJS({
           if (output.exitCode === 0) {
             return output.stdout.trim();
           } else {
+            core2.warning("Qodana needs git history to extract merge-base. Please specify fetch-depth: 0 in your workflow.");
             return pr.base.sha;
           }
         }
@@ -135482,7 +135483,7 @@ ${comment_tag_pattern}`;
       return __awaiter3(this, arguments, void 0, function* (args, options = {}) {
         const originalIgnoreReturnCode = options.ignoreReturnCode;
         const result = yield exec.getExecOutput("git", args, Object.assign(Object.assign({}, options), { ignoreReturnCode: true }));
-        if (result.exitCode !== 0 && !originalIgnoreReturnCode) {
+        if (result.exitCode !== 0 && originalIgnoreReturnCode) {
           core2.warning(`Git command failed: git ${args.join(" ")}
 Exit code: ${result.exitCode}
 Stdout: ${result.stdout}

scan/src/utils.ts

@@ -121,6 +121,9 @@ async function getPrSha(): Promise<string> {
     if (output.exitCode === 0) {
       return output.stdout.trim()
     } else {
+      core.warning(
+        'Qodana needs git history to extract merge-base. Please specify fetch-depth: 0 in your workflow.'
+      )
       return pr.base.sha
     }
   }
@@ -660,7 +663,7 @@ async function gitOutput(
     ...options,
     ignoreReturnCode: true
   })
-  if (result.exitCode !== 0 && !originalIgnoreReturnCode) {
+  if (result.exitCode !== 0 && originalIgnoreReturnCode) {
     core.warning(
       `Git command failed: git ${args.join(' ')}\nExit code: ${result.exitCode}\nStdout: ${result.stdout}\nStderr: ${result.stderr}`
     )

vsts/QodanaScan/index.js

@@ -81361,6 +81361,11 @@ Stderr: ${result2.stderr}`);
         if (result2.stdout.startsWith("[command]")) {
           result2.stdout = result2.stdout.slice(result2.stdout.indexOf(" ") + 1).replace(args.join(" "), "").trim();
         }
+        if (result2.exitCode !== 0) {
+          tl2.warning(`Git command git ${args.join(" ")} failed:
+Stdout: ${result2.stdout}
+Stderr: ${result2.stderr}`);
+        }
         return result2;
       });
     }

vsts/src/utils.ts

@@ -315,6 +315,11 @@ async function gitOutput(
       .replace(args.join(' '), '')
       .trim()
   }
+  if (result.exitCode !== 0) {
+    tl.warning(
+      `Git command git ${args.join(' ')} failed:\nStdout: ${result.stdout}\nStderr: ${result.stderr}`
+    )
+  }
   return result
 }
 

vsts/vss-extension.dev.json

@@ -2,7 +2,7 @@
   "manifestVersion": 1,
   "id": "qodana-dev",
   "name": "Qodana (Dev)",
-  "version": "2025.2.334",
+  "version": "2025.2.335",
   "publisher": "JetBrains",
   "targets": [
     {