Skip to content

Commit

Permalink
Merge pull request #15146 from cakephp/memcache-service
Browse files Browse the repository at this point in the history
Use memcache docker image instead of apt package.
  • Loading branch information
markstory committed Nov 10, 2020
2 parents 71ad4ab + 1db878f commit 6a0f6e9
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions .github/workflows/ci.yml
Expand Up @@ -30,6 +30,10 @@ jobs:
image: redis
ports:
- 6379/tcp
memcached:
image: memcached
ports:
- 11211/tcp

steps:
- name: Setup MySQL latest
Expand Down Expand Up @@ -82,10 +86,6 @@ jobs:

- name: Install packages
run: |
# memcached installation fails without updating packages.
sudo apt update
sudo apt install memcached
sudo locale-gen da_DK.UTF-8
sudo locale-gen de_DE.UTF-8
Expand All @@ -104,6 +104,7 @@ jobs:
- name: Run PHPUnit
env:
REDIS_PORT: ${{ job.services.redis.ports['6379'] }}
MEMCACHED_PORT: ${{ job.services.memcached.ports['11211'] }}
run: |
if [[ ${{ matrix.db-type }} == 'sqlite' ]]; then export DB_URL='sqlite:///:memory:'; fi
if [[ ${{ matrix.db-type }} == 'mysql' && ${{ matrix.php-version }} == '7.2' ]]; then export DB_URL='mysql://root:root@127.0.0.1/cakephp?init[]=SET sql_mode = "STRICT_TRANS_TABLES,NO_ZERO_IN_DATE,NO_ZERO_DATE,ERROR_FOR_DIVISION_BY_ZERO,NO_ENGINE_SUBSTITUTION"'; fi
Expand Down

0 comments on commit 6a0f6e9

Please sign in to comment.