Skip to content

Commit

Permalink
CI: rename matrix options to more significant names
Browse files Browse the repository at this point in the history
Currently the test matrix diplays `tests (true`) and `tests (false)`.
This is not super informative and if forces one to dive into the
workflows file to figure out what it means.

With more descriptive names, the purpose of each test becomes more
apparent. Here we're using the value that we need to give to the TESTVIM
env variable when we run the vader test suite.
  • Loading branch information
lelutin committed Aug 28, 2024
1 parent c71560d commit b26de45
Showing 1 changed file with 4 additions and 8 deletions.
12 changes: 4 additions & 8 deletions .github/workflows/vader.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,21 +27,17 @@ jobs:
strategy:
fail-fast: false
matrix:
neovim: [false, true]
editor: ["vim", "nvim"]

steps:
- uses: actions/checkout@v2
- name: Setup Vim or Neovim
- name: Setup ${{ matrix.editor }}
uses: rhysd/action-setup-vim@v1
id: vim
with:
neovim: ${{ matrix.neovim }}
neovim: ${{ matrix.editor == 'nvim' }}
- name: Run Vader unit tests
run: |
if [ "${{ matrix.neovim }}" = "true" ]; then
TESTVIM=nvim
else
TESTVIM=vim
fi
TESTVIM=${{ matrix.editor }}
export TESTVIM
./test/run-tests.sh

0 comments on commit b26de45

Please sign in to comment.