Skip to content

Commit f3070c0

Browse files
authored
Merge pull request #195 from stronk7/bump_42_to_php_80_pg_13
Bump CI tests to use PHP 8.0 and PostgreSQL 13
2 parents d041d7c + b713b2b commit f3070c0

File tree

7 files changed

+33
-24
lines changed

7 files changed

+33
-24
lines changed

.github/workflows/test.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,13 +26,13 @@ jobs:
2626
run: make validate
2727

2828
citest:
29-
name: CI test
29+
name: Integration tests
3030
needs: selftest
3131
runs-on: ubuntu-22.04
3232

3333
services:
3434
postgres:
35-
image: postgres:12
35+
image: postgres:13
3636
env:
3737
POSTGRES_USER: 'postgres'
3838
POSTGRES_HOST_AUTH_METHOD: 'trust'
@@ -45,7 +45,7 @@ jobs:
4545
fail-fast: false
4646
matrix:
4747
include:
48-
- php: '7.4'
48+
- php: '8.0'
4949
moodle-branch: 'master'
5050
- php: '7.4'
5151
moodle-branch: 'MOODLE_401_STABLE'
@@ -69,6 +69,7 @@ jobs:
6969
with:
7070
php-version: ${{ matrix.php }}
7171
extensions: pgsql, zip, gd, xmlrpc, soap
72+
ini-values: max_input_vars=5000
7273
# We want to verify that xdebug works for coverage. Once we only support
7374
# Moodle 3.10/PHPUnit 8 and up, we can switch our tests to pcov too.
7475
coverage: xdebug

.travis.dist.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
language: php
22

33
addons:
4-
postgresql: "12"
4+
postgresql: "13"
55
apt:
66
packages:
7-
- postgresql-12
8-
- postgresql-client-12
7+
- postgresql-13
8+
- postgresql-client-13
99

1010
services:
1111
- mysql
@@ -23,7 +23,7 @@ php:
2323

2424
env:
2525
global:
26-
- PGVER=12
26+
- PGVER=13
2727
- MOODLE_BRANCH=MOODLE_311_STABLE
2828
matrix:
2929
- DB=pgsql

.travis.yml

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
language: php
22

33
addons:
4-
postgresql: "12"
4+
postgresql: "13"
55
apt:
66
packages:
7-
- postgresql-12
8-
- postgresql-client-12
7+
- postgresql-13
8+
- postgresql-client-13
99

1010
services:
1111
- mysql
@@ -16,19 +16,16 @@ cache:
1616
- $HOME/.composer/cache
1717
- $HOME/.npm
1818

19-
php:
20-
- 7.4
21-
2219
env:
2320
global:
24-
- PGVER=12
21+
- PGVER=13
2522
- IGNORE_PATHS=ignore
2623
- IGNORE_NAMES=ignore_name.php
2724
- MUSTACHE_IGNORE_NAMES=broken.mustache
2825
- DB=pgsql
29-
- MOODLE_BRANCH=master
3026

3127
before_install:
28+
- echo 'max_input_vars=5000' >> ~/.phpenv/versions/$(phpenv version-name)/etc/conf.d/travis.ini
3229
# We want to verify that xdebug works for coverage. Once we only support
3330
# Moodle 3.10/PHPUnit 8 and up, we can switch our tests to pcov too.
3431
# - phpenv config-rm xdebug.ini
@@ -49,8 +46,6 @@ install:
4946
- moodle-plugin-ci install -vvv
5047

5148
script:
52-
- make test-phpunit
53-
5449
# This is basically "integration" testing.
5550
- moodle-plugin-ci phplint
5651
- moodle-plugin-ci phpcpd
@@ -67,14 +62,18 @@ script:
6762

6863
jobs:
6964
include:
70-
- stage: Tests
65+
- stage: CI test (make validate)
7166
php: 7.3
7267
before_install: skip
7368
install:
7469
- make init
7570
script:
7671
- make validate
72+
7773
- stage: Integration tests
74+
if: env(MOODLE_BRANCH) IS present
75+
- php: 8.0
76+
env: MOODLE_BRANCH=master
7877
- php: 7.4
7978
env: MOODLE_BRANCH=MOODLE_401_STABLE
8079
- php: 7.4

docs/CHANGELOG.md

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,15 @@ This project adheres to [Semantic Versioning](http://semver.org/).
99
The format of this change log follows the advice given at [Keep a CHANGELOG](http://keepachangelog.com).
1010

1111
## [Unreleased]
12+
### Changed
13+
- Updated [gha.dist.yml](https://github.com/moodlehq/moodle-plugin-ci/blob/master/gha.dist.yml) and
14+
[.travis.dist.yml](https://github.com/moodlehq/moodle-plugin-ci/blob/master/.travis.dist.yml)
15+
(and documentation) to fulfil [Moodle 4.2 new requirements](https://tracker.moodle.org/browse/MDL-74905).
16+
- ACTION REQUIRED: Review existing integrations running tests against master (4.2dev). There are a few Moodle 4.2 new requirements:
17+
- PHP 8.0 is required (instead of 7.4).
18+
- PostgreSQL 13 is required (instead of 12).
19+
- MariaDB 10.6 is required (instead of 10.4).
20+
- MySQL 8 is required (instead of 5.7).
1221

1322
## [3.4.2] - 2022-10-18
1423
### Changed

docs/GHAFileExplained.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ jobs:
3131
# DB services you need for testing.
3232
services:
3333
postgres:
34-
image: postgres:12
34+
image: postgres:13
3535
env:
3636
POSTGRES_USER: 'postgres'
3737
POSTGRES_HOST_AUTH_METHOD: 'trust'

docs/TravisFileExplained.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,11 +13,11 @@ language: php
1313

1414
# Installs the updated version of PostgreSQL and extra APT packages.
1515
addons:
16-
postgresql: "12"
16+
postgresql: "13"
1717
apt:
1818
packages:
19-
- postgresql-12
20-
- postgresql-client-12
19+
- postgresql-13
20+
- postgresql-client-13
2121

2222
# Ensure DB and docker services are running.
2323
services:
@@ -45,7 +45,7 @@ env:
4545
# used, because, for PG 11 and up, both the user and the port were
4646
# changed by Travis. With that variable, the tool will switch to
4747
# socketed connections instead of localhost ones.
48-
- PGVER=12
48+
- PGVER=13
4949
# This line determines which version branch of Moodle to test against.
5050
- MOODLE_BRANCH=MOODLE_311_STABLE
5151
# This matrix is used for testing against multiple databases. So for

gha.dist.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ jobs:
88

99
services:
1010
postgres:
11-
image: postgres:12
11+
image: postgres:13
1212
env:
1313
POSTGRES_USER: 'postgres'
1414
POSTGRES_HOST_AUTH_METHOD: 'trust'

0 commit comments

Comments
 (0)