Disable no-return-await rule in ESLint configuration and remove unnecessary comment in runCLICommand function

(#5092, gitkraken/vscode-gitlens-private#78)

Author: sergeibbb

eslint.config.mjs

@@ -218,7 +218,7 @@ export default defineConfig(
 			'no-restricted-globals': ['error', 'process'],
 			'no-restricted-imports': 'off',
 			'no-return-assign': 'error',
-			'no-return-await': 'warn',
+			'no-return-await': 'off', // Disabled in favor of @typescript-eslint/return-await (handles `using` correctly)
 			'no-self-compare': 'error',
 			'no-sequences': 'error',
 			'no-template-curly-in-string': 'warn',

src/env/node/gk/cli/utils.ts

@@ -118,7 +118,6 @@ export async function runCLICommand(args: string[], options?: { cwd?: string }):
 		throw new Error('CLI is not installed');
 	}
 
-	// eslint-disable-next-line no-return-await -- await is needed for the scope to be properly disposed
 	return await run(command, args, 'utf8', { cwd: cwd });
 }