From c91163725c390ee5a817f65bb6f9fb2e62938ee2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andr=C3=A9=20Carneiro?= Date: Mon, 3 Jul 2023 16:39:50 -0300 Subject: [PATCH] chore: use output file instead of environment variable --- .github/workflows/main.yml | 7 ++----- extras/github/docker.py | 9 +++------ 2 files changed, 5 insertions(+), 11 deletions(-) diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml index 0109f2bfe..4347c377f 100644 --- a/.github/workflows/main.yml +++ b/.github/workflows/main.yml @@ -38,11 +38,8 @@ jobs: matrix = full_matrix else: matrix = reduced_matrix - outputs = [] - if os.environ['GITHUB_OUTPUT']: - outputs.append(os.environ['GITHUB_OUTPUT']) - outputs.append('{name}={value}'.format(name='matrix', value=json.dumps(matrix))) - os.environ['GITHUB_OUTPUT'] = '\n'.join(outputs) + with open(os.environ['GITHUB_OUTPUT'], 'a') as f: + f.write('matrix={}\n'.format(json.dumps(matrix))) check-matrix: runs-on: ubuntu-latest needs: matrix diff --git a/extras/github/docker.py b/extras/github/docker.py index 28b6781c2..f1702b81d 100644 --- a/extras/github/docker.py +++ b/extras/github/docker.py @@ -3,12 +3,9 @@ from typing import Dict def print_output(output: Dict): - outputs = [] - if os.environ['GITHUB_OUTPUT']: - outputs.append(os.environ['GITHUB_OUTPUT']) - for k, v in output.items(): - outputs.append('{name}={value}'.format(name=k, value=v)) - os.environ['GITHUB_OUTPUT'] += '\n'.join(outputs) + outputs = ['{}={}\n'.format(k, v) for k, v in output.items()] + with open(os.environ['GITHUB_OUTPUT'], 'a') as f: + f.writelines(outputs) def prep_base_version(environ: Dict): GITHUB_REF = environ.get('GITHUB_REF')