Skip to content

Commit c644ed5

Browse files
committed
fix: adjust gh actions, remove psalm where it's not needed, it requires php >= 8.2
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
1 parent 0e9b1f2 commit c644ed5

3 files changed

Lines changed: 12 additions & 3 deletions

File tree

.github/workflows/integration.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,10 @@ jobs:
101101

102102
- name: Set up PHPUnit
103103
working-directory: apps/${{ env.APP_NAME }}
104-
run: composer i
104+
# remove psalm because it requires php >= 8.2 and we want to test with 8.1
105+
run: |
106+
composer uninstall --dev psalm/phar
107+
composer i
105108
106109
- name: Set up Nextcloud
107110
run: |

.github/workflows/lint.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,10 @@ jobs:
5050
coverage: none
5151

5252
- name: Install dependencies
53-
run: composer i
53+
# remove psalm because it requires php >= 8.2 and we want to test with 8.0
54+
run: |
55+
composer uninstall --dev psalm/phar
56+
composer i
5457
5558
- name: Run coding standards check
5659
run: composer run cs:check || ( echo 'Please run `composer run cs:fix` to format your code' && exit 1 )

.github/workflows/phpunit.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,10 @@ jobs:
100100

101101
- name: Set up PHPUnit
102102
working-directory: apps/${{ env.APP_NAME }}
103-
run: composer i
103+
# remove psalm because it requires php >= 8.2 and we want to test with 8.1
104+
run: |
105+
composer uninstall --dev psalm/phar
106+
composer i
104107
105108
- name: Set up Nextcloud
106109
run: |

0 commit comments

Comments
 (0)