Skip to content

Commit cac654d

Browse files
committed
Change to git file permissions error on build
1 parent 6e81c86 commit cac654d

1 file changed

Lines changed: 28 additions & 11 deletions

File tree

.github/workflows/pr.yml

Lines changed: 28 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -23,17 +23,19 @@ jobs:
2323
runs-on: ubuntu-latest
2424
steps:
2525
- uses: actions/checkout@v4
26+
27+
- name: Fix ownership (container)
28+
run: |
29+
docker compose run --rm --user 0 -e HOST_UID=$(id -u) -e HOST_GID=$(id -g) php bash -lc 'chown -R ${HOST_UID}:${HOST_GID} /app'
30+
2631
- name: Validate composer files
2732
run: |
28-
# Note that we don't use the --strict flag on validate due to the
29-
# package drupal/config_entity_revisions 2.0.x-dev being considered a
30-
# version cf.
31-
# https://getcomposer.org/doc/articles/versions.md#branches
32-
docker compose run --rm php composer validate composer.json
33+
docker compose run --rm --user "$(id -u):$(id -g)" php composer validate composer.json
34+
3335
- name: Check that composer file is normalized
3436
run: |
35-
docker compose run --rm php composer install
36-
docker compose run --rm php composer normalize --dry-run
37+
docker compose run --rm --user "$(id -u):$(id -g)" php composer install
38+
docker compose run --rm --user "$(id -u):$(id -g)" php composer normalize --dry-run
3739
3840
test-composer-files:
3941
name: Test composer files
@@ -43,29 +45,44 @@ jobs:
4345
dependency-version: [ prefer-lowest, prefer-stable ]
4446
steps:
4547
- uses: actions/checkout@v4
48+
49+
- name: Fix ownership (container)
50+
run: |
51+
docker compose run --rm --user 0 -e HOST_UID=$(id -u) -e HOST_GID=$(id -g) php bash -lc 'chown -R ${HOST_UID}:${HOST_GID} /app'
52+
4653
- name: Check that dependencies resolve.
4754
run: |
48-
# Clean up before update (cf. https://www.drupal.org/project/simplesamlphp_auth/issues/3350773)
4955
rm -fr vendor/
50-
docker compose run --rm php composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction
56+
docker compose run --rm --user "$(id -u):$(id -g)" php composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction
5157
5258
php-coding-standards:
5359
name: PHP coding standards
5460
runs-on: ubuntu-latest
5561
steps:
5662
- uses: actions/checkout@v4
63+
64+
- name: Fix ownership (container)
65+
run: |
66+
docker compose run --rm --user 0 -e HOST_UID=$(id -u) -e HOST_GID=$(id -g) php bash -lc 'chown -R ${HOST_UID}:${HOST_GID} /app'
67+
5768
- name: Install Dependencies
5869
run: |
59-
docker compose run --rm php composer install
70+
docker compose run --rm --user "$(id -u):$(id -g)" php composer install
71+
6072
- name: PHPCS
6173
run: |
62-
docker compose run --rm php composer coding-standards-check/phpcs
74+
docker compose run --rm --user "$(id -u):$(id -g)" php composer coding-standards-check/phpcs
6375
6476
php-code-analysis:
6577
name: PHP code analysis
6678
runs-on: ubuntu-latest
6779
steps:
6880
- uses: actions/checkout@v4
81+
82+
- name: Fix ownership (container)
83+
run: |
84+
docker compose run --rm --user 0 -e HOST_UID=$(id -u) -e HOST_GID=$(id -g) php bash -lc 'chown -R ${HOST_UID}:${HOST_GID} /app'
85+
6986
- name: Code analysis
7087
run: |
7188
./scripts/code-analysis

0 commit comments

Comments
 (0)