From 2ce7ea6b0c33febdce635f0deb5ccf8c3415d2f1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tom=C3=A1s=20Far=C3=ADas=20Santana?= Date: Tue, 7 Nov 2023 22:44:03 +0100 Subject: [PATCH] fix: Checkout master before hogql-diff and use new GH syntax (#18461) * fix: Replace deprecated set-output GH actions command * fix: Checkout master before checking for hogql changes * fix: But go back to branch after done * fix: Instead just diff with origin/master * fix: Let's just fetch manually * fix: Install libantlr, what's the worst that can happen? * fix: Install antlr the hogql-way * fix: Amazing hogql-developers already include this in wheel --- .github/actions/run-backend-tests/action.yml | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/.github/actions/run-backend-tests/action.yml b/.github/actions/run-backend-tests/action.yml index 45e189e23b1b6..9109dfc4eb9de 100644 --- a/.github/actions/run-backend-tests/action.yml +++ b/.github/actions/run-backend-tests/action.yml @@ -53,8 +53,9 @@ runs: shell: bash id: hogql-parser-diff run: | - changed=$(git diff --quiet HEAD master -- hogql_parser/ && echo "false" || echo "true") - echo "::set-output name=changed::$changed" + git fetch --no-tags --prune --depth=1 origin + changed=$(git diff --quiet HEAD origin/master -- hogql_parser/ && echo "false" || echo "true") + echo "changed=$changed" >> $GITHUB_OUTPUT - name: Install SAML (python3-saml) dependencies shell: bash