From 232c16745bbf245a4d0b36dd312ff61c16c71962 Mon Sep 17 00:00:00 2001 From: Andreas Arvidsson Date: Thu, 30 Jan 2025 17:25:38 +0100 Subject: [PATCH] Update test workflow for app_version variable (#2795) vscode_version was renamed to app_version, but some of the if checks were not updated ## Checklist - [ ] I have added [tests](https://www.cursorless.org/docs/contributing/test-case-recorder/) - [ ] I have updated the [docs](https://github.com/cursorless-dev/cursorless/tree/main/docs) and [cheatsheet](https://github.com/cursorless-dev/cursorless/tree/main/cursorless-talon/src/cheatsheet) - [ ] I have not broken the cheatsheet Co-authored-by: Phil Cohen --- .github/workflows/test.yml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 9d88d9fffa..dacdda46aa 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -49,13 +49,13 @@ jobs: - run: pnpm --color test if: runner.os != 'Linux' - run: xvfb-run -a pnpm -F @cursorless/test-harness test:talonJs - if: runner.os == 'Linux' && matrix.vscode_version == 'stable' + if: runner.os == 'Linux' && matrix.app_version == 'stable' - run: pnpm -F @cursorless/test-harness test:talonJs - if: runner.os != 'Linux' && matrix.vscode_version == 'stable' + if: runner.os != 'Linux' && matrix.app_version == 'stable' - run: xvfb-run -a pnpm -F @cursorless/cursorless-everywhere-talon-e2e test:quickjs - if: runner.os == 'Linux' && matrix.vscode_version == 'stable' + if: runner.os == 'Linux' && matrix.app_version == 'stable' - run: pnpm -F @cursorless/cursorless-everywhere-talon-e2e test:quickjs - if: runner.os != 'Linux' && matrix.vscode_version == 'stable' + if: runner.os != 'Linux' && matrix.app_version == 'stable' - run: bash -x scripts/install-neovim-dependencies.sh - uses: rhysd/action-setup-vim@v1 id: vim