Skip to content

Commit 900bb63

Browse files
authored
Merge pull request #170 from SimonFrings/ci
Use GitHub actions for continuous integration (CI)
2 parents 31b5641 + b555fca commit 900bb63

File tree

5 files changed

+52
-30
lines changed

5 files changed

+52
-30
lines changed

.gitattributes

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
/.gitattributes export-ignore
2+
/.github/ export-ignore
23
/.gitignore export-ignore
3-
/.travis.yml export-ignore
44
/examples export-ignore
55
/phpunit.xml.dist export-ignore
66
/phpunit.xml.legacy export-ignore

.github/workflows/ci.yml

Lines changed: 46 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,46 @@
1+
name: CI
2+
3+
on:
4+
push:
5+
pull_request:
6+
7+
jobs:
8+
PHPUnit:
9+
name: PHPUnit (PHP ${{ matrix.php }})
10+
runs-on: ubuntu-20.04
11+
strategy:
12+
matrix:
13+
php:
14+
- 8.0
15+
- 7.4
16+
- 7.3
17+
- 7.2
18+
- 7.1
19+
- 7.0
20+
- 5.6
21+
- 5.5
22+
- 5.4
23+
- 5.3
24+
steps:
25+
- uses: actions/checkout@v2
26+
- uses: shivammathur/setup-php@v2
27+
with:
28+
php-version: ${{ matrix.php }}
29+
coverage: xdebug
30+
- run: composer install
31+
- run: vendor/bin/phpunit --coverage-text
32+
if: ${{ matrix.php >= 7.3 }}
33+
- run: vendor/bin/phpunit --coverage-text -c phpunit.xml.legacy
34+
if: ${{ matrix.php < 7.3 }}
35+
36+
PHPUnit-hhvm:
37+
name: PHPUnit (HHVM)
38+
runs-on: ubuntu-18.04
39+
continue-on-error: true
40+
steps:
41+
- uses: actions/checkout@v2
42+
- uses: azjezz/setup-hhvm@v1
43+
with:
44+
version: lts-3.30
45+
- run: hhvm $(which composer) install
46+
- run: hhvm vendor/bin/phpunit

.travis.yml

Lines changed: 0 additions & 27 deletions
This file was deleted.

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Dns
22

3-
[![Build Status](https://travis-ci.org/reactphp/dns.svg?branch=master)](https://travis-ci.org/reactphp/dns)
3+
[![CI status](https://github.com/reactphp/dns/workflows/CI/badge.svg)](https://github.com/reactphp/dns/actions)
44

55
Async DNS resolver for [ReactPHP](https://reactphp.org/).
66

@@ -431,7 +431,7 @@ $ composer require react/dns:^1.4
431431
See also the [CHANGELOG](CHANGELOG.md) for details about version upgrades.
432432

433433
This project aims to run on any platform and thus does not require any PHP
434-
extensions and supports running on legacy PHP 5.3 through current PHP 7+ and
434+
extensions and supports running on legacy PHP 5.3 through current PHP 8+ and
435435
HHVM.
436436
It's *highly recommended to use PHP 7+* for this project.
437437

tests/FunctionalResolverTest.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -119,6 +119,9 @@ public function testResolveInvalidRejects()
119119

120120
public function testResolveCancelledRejectsImmediately()
121121
{
122+
// max_nesting_level was set to 100 for PHP Versions < 5.4 which resulted in failing test for legacy PHP
123+
ini_set('xdebug.max_nesting_level', 256);
124+
122125
$ex = $this->callback(function ($param) {
123126
return ($param instanceof \RuntimeException && $param->getMessage() === 'DNS query for google.com has been cancelled');
124127
});

0 commit comments

Comments
 (0)