From 4fefcac86d3680f345f34c2c52839e76f8a5ae38 Mon Sep 17 00:00:00 2001 From: Eshaan Aggarwal <96648934+EshaanAgg@users.noreply.github.com> Date: Sun, 24 Dec 2023 23:04:23 +0530 Subject: [PATCH] ci: simplify check-label workflow (#2068) ## Which problem is this PR solving? Fixes #2067 ## Description of the changes Removes the `set-output` command from the workflow ## How was this change tested? NA ## Checklist - [X] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [X] I have signed all commits - [X] I have added unit tests for the new functionality - [X] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: Eshaan Aggarwal <96648934+EshaanAgg@users.noreply.github.com> Co-authored-by: Yuri Shkuro --- .github/workflows/ci-label-check.yml | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/.github/workflows/ci-label-check.yml b/.github/workflows/ci-label-check.yml index d3897ffb0b..a024abaa3d 100644 --- a/.github/workflows/ci-label-check.yml +++ b/.github/workflows/ci-label-check.yml @@ -14,12 +14,8 @@ jobs: check-label: runs-on: ubuntu-latest steps: - - name: Check PR author - id: check_author - run: echo "::set-output name=is_dependabot::$(echo ${{ github.event.pull_request.user.login }} | grep -o 'dependabot')" - - name: Check PR label - if: steps.check_author.outputs.is_dependabot != 'dependabot' + if: github.event.pull_request.user.login != 'dependabot' run: | LABEL_NAME="changelog:" if [[ $(curl -s "https://api.github.com/repos/${{ github.repository }}/pulls/${{ github.event.pull_request.number }}" | jq -r '.labels[].name' | grep -c "^$LABEL_NAME") -eq 0 ]]; then