Skip to content

Commit f39404d

Browse files
ondrejmirtesondrejmirtes
committed
Use GitHub Actions instead of Travis CI
1 parent 70b84f6 commit f39404d

File tree

9 files changed

+194
-55
lines changed

9 files changed

+194
-55
lines changed

Diff for: ‎.github/workflows/build.yml

+160
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,160 @@
1+
# https://help.github.com/en/categories/automating-your-workflow-with-github-actions
2+
3+
name: "Build"
4+
5+
on:
6+
pull_request:
7+
push:
8+
branches:
9+
- "master"
10+
11+
jobs:
12+
lint:
13+
name: "Lint"
14+
runs-on: "ubuntu-latest"
15+
16+
strategy:
17+
matrix:
18+
php-version:
19+
- "7.1"
20+
- "7.2"
21+
- "7.3"
22+
- "7.4"
23+
- "8.0"
24+
25+
steps:
26+
- name: "Checkout"
27+
uses: "actions/checkout@v2"
28+
29+
- name: "Install PHP"
30+
uses: "shivammathur/setup-php@v2"
31+
with:
32+
coverage: "none"
33+
php-version: "${{ matrix.php-version }}"
34+
35+
- name: "Validate Composer"
36+
run: "composer validate"
37+
38+
- name: "Install dependencies"
39+
run: "composer install --no-interaction --no-progress --no-suggest"
40+
41+
- name: "Update PHPUnit"
42+
if: matrix.php-version == '7.4' || matrix.php-version == '8.0'
43+
run: "composer require --dev phpunit/phpunit:'^9.5' --update-with-dependencies"
44+
45+
46+
- name: "Lint"
47+
run: "vendor/bin/phing lint"
48+
49+
coding-standards:
50+
name: "Coding Standard"
51+
52+
runs-on: "ubuntu-latest"
53+
54+
steps:
55+
- name: "Checkout"
56+
uses: "actions/checkout@v2"
57+
58+
- name: "Install PHP"
59+
uses: "shivammathur/setup-php@v2"
60+
with:
61+
coverage: "none"
62+
php-version: "7.4"
63+
64+
- name: "Validate Composer"
65+
run: "composer validate"
66+
67+
- name: "Install dependencies"
68+
run: "composer install --no-interaction --no-progress --no-suggest"
69+
70+
- name: "Lint"
71+
run: "vendor/bin/phing lint"
72+
73+
- name: "Coding Standard"
74+
run: "vendor/bin/phing cs"
75+
76+
tests:
77+
name: "Tests"
78+
runs-on: "ubuntu-latest"
79+
80+
strategy:
81+
fail-fast: false
82+
matrix:
83+
php-version:
84+
- "7.1"
85+
- "7.2"
86+
- "7.3"
87+
- "7.4"
88+
- "8.0"
89+
dependencies:
90+
- "lowest"
91+
- "highest"
92+
93+
steps:
94+
- name: "Checkout"
95+
uses: "actions/checkout@v2"
96+
97+
- name: "Install PHP"
98+
uses: "shivammathur/setup-php@v2"
99+
with:
100+
coverage: "none"
101+
php-version: "${{ matrix.php-version }}"
102+
103+
- name: "Install lowest dependencies"
104+
if: ${{ matrix.dependencies == 'lowest' }}
105+
run: "composer update --prefer-lowest --no-interaction --no-progress --no-suggest"
106+
107+
- name: "Install highest dependencies"
108+
if: ${{ matrix.dependencies == 'highest' }}
109+
run: "composer update --no-interaction --no-progress --no-suggest"
110+
111+
- name: "Update PHPUnit"
112+
if: matrix.php-version == '7.4' || matrix.php-version == '8.0'
113+
run: "composer require --dev phpunit/phpunit:'^9.5' --update-with-dependencies"
114+
115+
- name: "Tests"
116+
run: "vendor/bin/phing tests"
117+
118+
static-analysis:
119+
name: "PHPStan"
120+
runs-on: "ubuntu-latest"
121+
122+
strategy:
123+
fail-fast: false
124+
matrix:
125+
php-version:
126+
- "7.1"
127+
- "7.2"
128+
- "7.3"
129+
- "7.4"
130+
- "8.0"
131+
dependencies:
132+
- "lowest"
133+
- "highest"
134+
135+
steps:
136+
- name: "Checkout"
137+
uses: "actions/checkout@v2"
138+
139+
- name: "Install PHP"
140+
uses: "shivammathur/setup-php@v2"
141+
with:
142+
coverage: "none"
143+
php-version: "${{ matrix.php-version }}"
144+
extensions: mbstring
145+
tools: composer:v2
146+
147+
- name: "Install lowest dependencies"
148+
if: ${{ matrix.dependencies == 'lowest' }}
149+
run: "composer update --prefer-lowest --no-interaction --no-progress --no-suggest"
150+
151+
- name: "Install highest dependencies"
152+
if: ${{ matrix.dependencies == 'highest' }}
153+
run: "composer update --no-interaction --no-progress --no-suggest"
154+
155+
- name: "Update PHPUnit"
156+
if: matrix.php-version == '7.4' || matrix.php-version == '8.0'
157+
run: "composer require --dev phpunit/phpunit:'^9.5' --update-with-dependencies"
158+
159+
- name: "PHPStan"
160+
run: "vendor/bin/phing phpstan"

Diff for: ‎.travis.yml

-15
This file was deleted.

Diff for: ‎README.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Extra strict and opinionated rules for PHPStan
22

3-
[![Build Status](https://travis-ci.com/phpstan/phpstan-strict-rules.svg?branch=master)](https://travis-ci.com/phpstan/phpstan-strict-rules)
3+
[![Build](https://github.com/phpstan/phpstan-strict-rules/workflows/Build/badge.svg)](https://github.com/phpstan/phpstan-strict-rules/actions)
44
[![Latest Stable Version](https://poser.pugx.org/phpstan/phpstan-strict-rules/v/stable)](https://packagist.org/packages/phpstan/phpstan-strict-rules)
55
[![License](https://poser.pugx.org/phpstan/phpstan-strict-rules/license)](https://packagist.org/packages/phpstan/phpstan-strict-rules)
66

Diff for: ‎build-cs/.gitignore

+2
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
/composer.lock
2+
/vendor

Diff for: ‎build-cs/composer.json

+7
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
{
2+
"require-dev": {
3+
"consistence/coding-standard": "^3.10",
4+
"dealerdirect/phpcodesniffer-composer-installer": "^0.7.0",
5+
"slevomat/coding-standard": "^6.4"
6+
}
7+
}

Diff for: ‎build.xml

+13-28
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
composer-install,
77
lint,
88
cs,
9-
composer-normalize-check,
109
tests,
1110
phpstan
1211
"/>
@@ -34,31 +33,6 @@
3433
</exec>
3534
</target>
3635

37-
<target name="composer-normalize-check">
38-
<exec
39-
executable="composer"
40-
logoutput="true"
41-
passthru="true"
42-
checkreturn="true"
43-
>
44-
<arg value="normalize"/>
45-
<arg value="--ansi"/>
46-
<arg value="--dry-run"/>
47-
</exec>
48-
</target>
49-
50-
<target name="composer-normalize-fix">
51-
<exec
52-
executable="composer"
53-
logoutput="true"
54-
passthru="true"
55-
checkreturn="true"
56-
>
57-
<arg value="normalize"/>
58-
<arg value="--ansi"/>
59-
</exec>
60-
</target>
61-
6236
<target name="lint">
6337
<exec
6438
executable="vendor/bin/parallel-lint"
@@ -75,7 +49,18 @@
7549

7650
<target name="cs">
7751
<exec
78-
executable="vendor/bin/phpcs"
52+
executable="composer"
53+
logoutput="true"
54+
passthru="true"
55+
checkreturn="true"
56+
>
57+
<arg value="install"/>
58+
<arg value="--working-dir"/>
59+
<arg path="build-cs"/>
60+
<arg value="--ansi"/>
61+
</exec>
62+
<exec
63+
executable="build-cs/vendor/bin/phpcs"
7964
logoutput="true"
8065
passthru="true"
8166
checkreturn="true"
@@ -91,7 +76,7 @@
9176

9277
<target name="cs-fix">
9378
<exec
94-
executable="vendor/bin/phpcbf"
79+
executable="build-cs/vendor/bin/phpcbf"
9580
logoutput="true"
9681
passthru="true"
9782
checkreturn="true"

Diff for: ‎composer.json

+7-8
Original file line numberDiff line numberDiff line change
@@ -7,19 +7,18 @@
77
],
88
"require": {
99
"php": "^7.1 || ^8.0",
10-
"phpstan/phpstan": "^0.12.33"
10+
"phpstan/phpstan": "^0.12.60"
1111
},
1212
"require-dev": {
13-
"consistence/coding-standard": "^3.10",
14-
"dealerdirect/phpcodesniffer-composer-installer": "^0.7.0",
15-
"ergebnis/composer-normalize": "^2.0.2",
16-
"phing/phing": "^2.16.0",
13+
"phing/phing": "^2.16.3",
1714
"php-parallel-lint/php-parallel-lint": "^1.2",
18-
"phpstan/phpstan-phpunit": "^0.12",
19-
"phpunit/phpunit": "^7.0",
20-
"slevomat/coding-standard": "^6.4"
15+
"phpstan/phpstan-phpunit": "^0.12.16",
16+
"phpunit/phpunit": "^7.5.20"
2117
},
2218
"config": {
19+
"platform": {
20+
"php": "7.4.6"
21+
},
2322
"sort-packages": true
2423
},
2524
"extra": {

Diff for: ‎phpcs.xml

+3-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
<?xml version="1.0"?>
2-
<ruleset name="PHPStan strict rules">
3-
<rule ref="vendor/consistence/coding-standard/Consistence/ruleset.xml">
2+
<ruleset name="PHPStan Strict Rules extension">
3+
<rule ref="build-cs/vendor/consistence/coding-standard/Consistence/ruleset.xml">
44
<exclude name="Squiz.Functions.GlobalFunction.Found"/>
55
<exclude name="SlevomatCodingStandard.Namespaces.FullyQualifiedClassNameAfterKeyword"/>
66
<exclude name="SlevomatCodingStandard.Namespaces.FullyQualifiedClassNameInAnnotation"/>
@@ -23,6 +23,7 @@
2323
@dataProvider,
2424
@requires
2525
"/>
26+
<property name="enableObjectTypeHint" value="false"/>
2627
</properties>
2728
<exclude name="SlevomatCodingStandard.TypeHints.TypeHintDeclaration.MissingTraversableParameterTypeHintSpecification"/>
2829
<exclude name="SlevomatCodingStandard.TypeHints.TypeHintDeclaration.MissingTraversableReturnTypeHintSpecification"/>

Diff for: ‎tests/Rules/Classes/data/call-to-parent-constructor-php-lt-74.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
<?php
1+
<?php // lint <= 7.4
22

33
class CCallToParentConstructor
44
{

0 commit comments

Comments
 (0)