Skip to content

Commit

Permalink
Rename test_type to test-type
Browse files Browse the repository at this point in the history
Since kebab-case is more common than snake_case for this usage, and
this convention was otherwise being used throughout the workflows.
  • Loading branch information
EliahKagan committed Sep 17, 2023
1 parent ff16cf6 commit b3b3d0a
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
- "3.10"
- "3.11"
- "3.12"
test_type:
test-type:
- core
include:
- experimental: false
Expand All @@ -32,18 +32,18 @@ jobs:
python-version: ${{ matrix.python-version }}
python-allow-prereleases: ${{ matrix.experimental }}
poetry-version: "1.6.1"
cache-key: ${{ matrix.test_type }}
cache-key: ${{ matrix.test-type }}
install-command: |
if [ "${{ matrix.test_type }}" == "core" ]; then
if [ "${{ matrix.test-type }}" == "core" ]; then
echo "Running core tests, installing dependencies with poetry..."
poetry install --only=main,test
else
echo "Running extended tests, installing dependencies with poetry..."
poetry install --only=main,test --all-extras
fi
- name: Run ${{matrix.test_type}} tests
- name: Run ${{matrix.test-type}} tests
run: |
if [ "${{ matrix.test_type }}" == "core" ]; then
if [ "${{ matrix.test-type }}" == "core" ]; then
make test
else
make extended_tests
Expand Down

0 comments on commit b3b3d0a

Please sign in to comment.