Skip to content

Commit 3bb15ad

Browse files
committed
[TASK] Drop the highest/lowest Composer version switch
As we don't have any direct production dependencies, it does not make sense to test both with highest and lowest Composer versions on CI. We accidentally added this highest/lowest switch with the new coverage job. This commit removes this again, allowing for sharing the Composer caches with the main CI job.
1 parent 8ccd0bb commit 3bb15ad

File tree

1 file changed

+3
-11
lines changed

1 file changed

+3
-11
lines changed

.github/workflows/codecoverage.yml

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,6 @@ jobs:
1919
matrix:
2020
php-version:
2121
- '7.4'
22-
dependencies:
23-
- highest
2422

2523
steps:
2624
- name: Checkout
@@ -44,19 +42,13 @@ jobs:
4442
uses: actions/cache@v4
4543
with:
4644
path: ~/.cache/composer
47-
key: php${{ matrix.php-version }}-${{ matrix.dependencies }}-composer-${{ hashFiles('**/composer.json') }}
45+
key: php${{ matrix.php-version }}-composer-${{ hashFiles('**/composer.json') }}
4846
restore-keys: |
49-
php${{ matrix.php-version }}-${{ matrix.dependencies }}-composer-
47+
php${{ matrix.php-version }}-composer-
5048
5149
- name: Install Composer dependencies
5250
run: |
53-
if [[ "${{ matrix.dependencies }}" == 'lowest' ]]; then
54-
DEPENDENCIES='--prefer-lowest';
55-
else
56-
DEPENDENCIES='';
57-
fi;
58-
composer install --no-progress;
59-
composer update --with-dependencies --no-progress "${DEPENDENCIES}";
51+
composer update --with-dependencies --no-progress;
6052
composer show;
6153
6254
- name: Run Tests

0 commit comments

Comments
 (0)