Skip to content

Commit

Permalink
Update test workflow for app_version variable (#2795)
Browse files Browse the repository at this point in the history
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 <phillip@phillip.io>
  • Loading branch information
AndreasArvidsson and phillco authored Jan 30, 2025
1 parent 5338825 commit 232c167
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 232c167

Please sign in to comment.