Skip to content

Commit ba78d3e

Browse files
nextcloud-botnickvergessen
authored andcommitted
chore(CI): Updating pr-feedback.yml workflow from template
Signed-off-by: Nextcloud bot <[email protected]>
1 parent 97a5ec5 commit ba78d3e

12 files changed

+36
-15
lines changed

.github/workflows/appstore-build-publish.yml

+3
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,9 @@ on:
1212
release:
1313
types: [published]
1414

15+
permissions:
16+
contents: write
17+
1518
jobs:
1619
build_and_publish:
1720
runs-on: ubuntu-latest

.github/workflows/dependabot-approve-merge.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
name: Dependabot
1010

1111
on:
12-
pull_request_target: # zizmor: ignore[dangerous-triggers]
12+
pull_request:
1313
branches:
1414
- main
1515
- master
@@ -24,7 +24,7 @@ concurrency:
2424

2525
jobs:
2626
auto-approve-merge:
27-
if: github.actor == 'dependabot[bot]' || github.actor == 'renovate[bot]'
27+
if: github.event.pull_request.user.login == 'dependabot[bot]' || github.event.pull_request.user.login == 'renovate[bot]'
2828
runs-on: ubuntu-latest-low
2929
permissions:
3030
# for hmarr/auto-approve-action to approve PRs

.github/workflows/integration.yml

+7-2
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,9 @@ on:
1010
- master
1111
- stable*
1212

13+
permissions:
14+
contents: read
15+
1316
env:
1417
APP_NAME: files_accesscontrol
1518

@@ -54,8 +57,9 @@ jobs:
5457

5558
steps:
5659
- name: Checkout server
57-
uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
60+
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
5861
with:
62+
persist-credentials: false
5963
repository: nextcloud/server
6064
ref: ${{ matrix.server-versions }}
6165

@@ -68,8 +72,9 @@ jobs:
6872
cd build/integration && composer require --dev phpunit/phpunit:~9
6973
7074
- name: Checkout app
71-
uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
75+
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
7276
with:
77+
persist-credentials: false
7378
path: apps/${{ env.APP_NAME }}
7479

7580
- name: Set up php ${{ matrix.php-versions }}

.github/workflows/phpunit-mariadb.yml

+3-3
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ jobs:
7070
matrix:
7171
php-versions: ${{ fromJson(needs.matrix.outputs.php-version) }}
7272
server-versions: ${{ fromJson(needs.matrix.outputs.server-max) }}
73-
mariadb-versions: ['10.6', '10.11']
73+
mariadb-versions: ['10.6', '11.4']
7474

7575
name: MariaDB ${{ matrix.mariadb-versions }} PHP ${{ matrix.php-versions }} Nextcloud ${{ matrix.server-versions }}
7676

@@ -80,8 +80,8 @@ jobs:
8080
ports:
8181
- 4444:3306/tcp
8282
env:
83-
MYSQL_ROOT_PASSWORD: rootpassword
84-
options: --health-cmd="mysqladmin ping" --health-interval 5s --health-timeout 2s --health-retries 5
83+
MARIADB_ROOT_PASSWORD: rootpassword
84+
options: --health-cmd="mariadb-admin ping" --health-interval 5s --health-timeout 2s --health-retries 5
8585

8686
steps:
8787
- name: Set app env

.github/workflows/phpunit-mysql.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -106,11 +106,11 @@ jobs:
106106
with:
107107
php-version: ${{ matrix.php-versions }}
108108
# https://docs.nextcloud.com/server/stable/admin_manual/installation/source_installation.html#prerequisites-for-manual-installation
109-
extensions: apcu, bz2, ctype, curl, dom, fileinfo, gd, iconv, intl, json, libxml, mbstring, openssl, pcntl, posix, session, simplexml, xmlreader, xmlwriter, zip, zlib, mysql, pdo_mysql
109+
extensions: bz2, ctype, curl, dom, fileinfo, gd, iconv, intl, json, libxml, mbstring, openssl, pcntl, posix, session, simplexml, xmlreader, xmlwriter, zip, zlib, mysql, pdo_mysql
110110
coverage: none
111111
ini-file: development
112112
# Temporary workaround for missing pcntl_* in PHP 8.3
113-
ini-values: apc.enable_cli=on, disable_functions=
113+
ini-values: disable_functions=
114114
env:
115115
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
116116

.github/workflows/phpunit-oci.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -119,11 +119,11 @@ jobs:
119119
with:
120120
php-version: ${{ matrix.php-versions }}
121121
# https://docs.nextcloud.com/server/stable/admin_manual/installation/source_installation.html#prerequisites-for-manual-installation
122-
extensions: apcu, bz2, ctype, curl, dom, fileinfo, gd, iconv, intl, json, libxml, mbstring, openssl, pcntl, posix, session, simplexml, xmlreader, xmlwriter, zip, zlib, oci8
122+
extensions: bz2, ctype, curl, dom, fileinfo, gd, iconv, intl, json, libxml, mbstring, openssl, pcntl, posix, session, simplexml, xmlreader, xmlwriter, zip, zlib, oci8
123123
coverage: none
124124
ini-file: development
125125
# Temporary workaround for missing pcntl_* in PHP 8.3
126-
ini-values: apc.enable_cli=on, disable_functions=
126+
ini-values: disable_functions=
127127
env:
128128
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
129129

.github/workflows/phpunit-pgsql.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -109,11 +109,11 @@ jobs:
109109
with:
110110
php-version: ${{ matrix.php-versions }}
111111
# https://docs.nextcloud.com/server/stable/admin_manual/installation/source_installation.html#prerequisites-for-manual-installation
112-
extensions: apcu, bz2, ctype, curl, dom, fileinfo, gd, iconv, intl, json, libxml, mbstring, openssl, pcntl, posix, session, simplexml, xmlreader, xmlwriter, zip, zlib, pgsql, pdo_pgsql
112+
extensions: bz2, ctype, curl, dom, fileinfo, gd, iconv, intl, json, libxml, mbstring, openssl, pcntl, posix, session, simplexml, xmlreader, xmlwriter, zip, zlib, pgsql, pdo_pgsql
113113
coverage: none
114114
ini-file: development
115115
# Temporary workaround for missing pcntl_* in PHP 8.3
116-
ini-values: apc.enable_cli=on, disable_functions=
116+
ini-values: disable_functions=
117117
env:
118118
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
119119

.github/workflows/phpunit-sqlite.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -98,11 +98,11 @@ jobs:
9898
with:
9999
php-version: ${{ matrix.php-versions }}
100100
# https://docs.nextcloud.com/server/stable/admin_manual/installation/source_installation.html#prerequisites-for-manual-installation
101-
extensions: apcu, bz2, ctype, curl, dom, fileinfo, gd, iconv, intl, json, libxml, mbstring, openssl, pcntl, posix, session, simplexml, xmlreader, xmlwriter, zip, zlib, sqlite, pdo_sqlite
101+
extensions: bz2, ctype, curl, dom, fileinfo, gd, iconv, intl, json, libxml, mbstring, openssl, pcntl, posix, session, simplexml, xmlreader, xmlwriter, zip, zlib, sqlite, pdo_sqlite
102102
coverage: none
103103
ini-file: development
104104
# Temporary workaround for missing pcntl_* in PHP 8.3
105-
ini-values: apc.enable_cli=on, disable_functions=
105+
ini-values: disable_functions=
106106
env:
107107
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
108108

.github/workflows/pr-feedback.yml

+4
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,10 @@ on:
1515
schedule:
1616
- cron: '30 1 * * *'
1717

18+
permissions:
19+
contents: read
20+
pull-requests: write
21+
1822
jobs:
1923
pr-feedback:
2024
if: ${{ github.repository_owner == 'nextcloud' }}

.github/workflows/psalm.yml

+3
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,9 @@ concurrency:
1414
group: psalm-${{ github.head_ref || github.run_id }}
1515
cancel-in-progress: true
1616

17+
permissions:
18+
contents: read
19+
1720
jobs:
1821
static-analysis:
1922
runs-on: ubuntu-latest

.github/workflows/reuse.yml

+3
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,9 @@ name: REUSE Compliance Check
1111

1212
on: [pull_request]
1313

14+
permissions:
15+
contents: read
16+
1417
jobs:
1518
reuse-compliance-check:
1619
runs-on: ubuntu-latest

.github/workflows/update-nextcloud-ocp.yml

+3
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,9 @@ on:
1313
schedule:
1414
- cron: "5 2 * * 0"
1515

16+
permissions:
17+
contents: read
18+
1619
jobs:
1720
update-nextcloud-ocp:
1821
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)