Skip to content

Commit 09d35ce

Browse files
authored
[TASK] Drop the highest/lowest Composer version switch (#698)
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 09d35ce

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)