Merge pull request #6683 from tamasvajk/feature/csv-coverage-fix

Only leave CSV coverage updater job enabled on github/codeql
This commit is contained in:
Tamás Vajk
2021-09-14 10:13:28 +02:00
committed by GitHub
2 changed files with 11 additions and 2 deletions

View File

@@ -8,7 +8,7 @@ on:
jobs:
update:
name: Update framework coverage report
if: github.event.repository.fork == false
if: github.repository == 'github/codeql'
runs-on: ubuntu-latest
steps:

View File

@@ -8,7 +8,16 @@ import sys
def subprocess_run(cmd):
"""Runs a command through subprocess.run, with a few tweaks. Raises an Exception if exit code != 0."""
print(shlex.join(cmd))
return subprocess.run(cmd, capture_output=True, text=True, env=os.environ.copy(), check=True)
try:
ret = subprocess.run(cmd, capture_output=True,
text=True, env=os.environ.copy(), check=True)
if (ret.stdout):
print(ret.stdout)
return ret
except subprocess.CalledProcessError as e:
if (e.stderr):
print(e.stderr)
raise e
def subprocess_check_output(cmd):