Skip to content

Commit

Permalink
fix: Checkout master before hogql-diff and use new GH syntax (#18461)
Browse files Browse the repository at this point in the history
* 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
  • Loading branch information
tomasfarias authored Nov 7, 2023
1 parent df1d102 commit 2ce7ea6
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions .github/actions/run-backend-tests/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 2ce7ea6

Please sign in to comment.