Skip to content

Commit c9d33f4

Browse files
committed
Merge branch 'master' into in-portal
* master: Drop support for PHP 5.4 and PHP 5.5 (#109)
2 parents 26b67b4 + 7fdbc74 commit c9d33f4

File tree

4 files changed

+679
-146
lines changed

4 files changed

+679
-146
lines changed

.github/workflows/tests.yml

Lines changed: 22 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -16,23 +16,25 @@ defaults:
1616
jobs:
1717

1818
tests:
19-
name: Tests (PHP ${{ matrix.php }}, PHP_CodeSniffer ${{ matrix.phpcs_version }})
20-
runs-on: Ubuntu-20.04
19+
name: "PHP ${{ matrix.php }}${{ matrix.with_coverage == true && ' with coverage' || ''}}, PHPCS ${{ matrix.phpcs_version }}"
20+
runs-on: ubuntu-latest
2121

2222
strategy:
2323
matrix:
24-
php: [ '5.4', '5.5', '5.6', '7.0', '7.1', '7.2', '7.3', '7.4']
25-
phpcs_version: ['^3.3']
24+
php: [ '5.6', '7.0', '7.1', '7.2', '7.3', '7.4']
25+
with_coverage: [ false ]
26+
phpcs_version: ['^3.9']
2627
include:
27-
- php: 5.4
28-
phpcs_version: ^3.3@dev
28+
- php: 7.4
29+
phpcs_version: ^3.9@dev
2930
- php: 7.2
30-
phpcs_version: ^3.3@dev
31+
phpcs_version: ^3.9
32+
with_coverage: true
3133
fail-fast: false
3234

3335
steps:
3436
- name: Checkout
35-
uses: actions/checkout@v2
37+
uses: actions/checkout@v4
3638
with:
3739
fetch-depth: 2
3840

@@ -48,16 +50,25 @@ jobs:
4850
composer require squizlabs/php_codesniffer:${{ matrix.phpcs_version }} --dev --no-update
4951
5052
- name: Install dependencies
51-
uses: "ramsey/composer-install@v1"
53+
uses: "ramsey/composer-install@v3"
5254
with:
5355
dependency-versions: "highest"
5456

55-
- name: Run tests
57+
- name: Run tests with Coverage
58+
if: "${{ matrix.with_coverage == true }}"
5659
run: |
5760
vendor/bin/phpunit -v --coverage-clover=coverage.clover
5861
62+
- name: Run tests without Coverage
63+
if: "${{ matrix.with_coverage == false }}"
64+
run: |
65+
vendor/bin/phpunit -v
66+
5967
- name: Upload Coverage to CodeCov
60-
uses: codecov/codecov-action@v2
68+
if: "${{ matrix.with_coverage == true }}"
69+
uses: codecov/codecov-action@v4
70+
with:
71+
token: ${{ secrets.CODECOV_TOKEN }}
6172

6273
- name: Upload Coverage to Scrutinizer CI (PHP < 8.0)
6374
if: "${{ matrix.php < '8.0' }}"

.phpunit.result.cache

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
{"version":1,"defects":[],"times":{"CodingStandard\\Tests\\Arrays\\ArrayUnitTest::testSniff":0.014,"CodingStandard\\Tests\\Classes\\ClassCreateInstanceUnitTest::testSniff":0.003,"CodingStandard\\Tests\\Classes\\ClassDeclarationUnitTest::testSniff":0.026,"CodingStandard\\Tests\\Classes\\ClassNamespaceUnitTest::testSniff":0,"CodingStandard\\Tests\\Classes\\PropertyDeclarationUnitTest::testSniff":0.001,"CodingStandard\\Tests\\CodeAnalysis\\WrongParentCallUnitTest::testSniff":0.001,"CodingStandard\\Tests\\Commenting\\DocCommentUnitTest::testSniff":0.024,"CodingStandard\\Tests\\Commenting\\FunctionCommentUnitTest::testSniff":0.102,"CodingStandard\\Tests\\Commenting\\InlineCommentUnitTest::testSniff":0.02,"CodingStandard\\Tests\\Commenting\\TypeCommentUnitTest::testSniff":0.018,"CodingStandard\\Tests\\ControlStructures\\AssignmentInConditionUnitTest::testSniff":0.001,"CodingStandard\\Tests\\ControlStructures\\ControlSignatureUnitTest::testSniff":0.005,"CodingStandard\\Tests\\Files\\OneTraitPerFileUnitTest::testSniff":0,"CodingStandard\\Tests\\Formatting\\BlankLineBeforeReturnUnitTest::testSniff":0.003,"CodingStandard\\Tests\\Formatting\\ItemAssignmentUnitTest::testSniff":0.004,"CodingStandard\\Tests\\Formatting\\NamespaceDeclarationUnitTest::testSniff":0.005,"CodingStandard\\Tests\\Formatting\\NoSpaceAfterBooleanNotUnitTest::testSniff":0.002,"CodingStandard\\Tests\\Formatting\\SpaceOperatorUnitTest::testSniff":0.007,"CodingStandard\\Tests\\Formatting\\SpaceUnaryOperatorUnitTest::testSniff":0.002,"CodingStandard\\Tests\\NamingConventions\\ValidClassNameUnitTest::testSniff":0,"CodingStandard\\Tests\\NamingConventions\\ValidFunctionNameUnitTest::testSniff":0.012,"CodingStandard\\Tests\\NamingConventions\\ValidInterfaceNameUnitTest::testSniff":0,"CodingStandard\\Tests\\NamingConventions\\ValidTraitNameUnitTest::testSniff":0,"CodingStandard\\Tests\\NamingConventions\\ValidVariableNameUnitTest::testSniff":0.005,"CodingStandard\\Tests\\PHP\\NoSilencedErrorsUnitTest::testSniff":0.001,"CodingStandard\\Tests\\Strings\\ConcatenationSpacingUnitTest::testSniff":0.004,"CodingStandard\\Tests\\WhiteSpace\\CommaSpacingUnitTest::testSniff":0.003,"CodingStandard\\Tests\\WhiteSpace\\ControlStructureSpacingUnitTest::testSniff":0.142,"CodingStandard\\Tests\\WhiteSpace\\ObjectOperatorSpacingUnitTest::testSniff":0.002}}

CodingStandard/Tests/Commenting/FunctionCommentUnitTest.php

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,8 +68,7 @@ public function getErrorList($testFile)
6868
138 => 4,
6969
139 => 4,
7070
143 => 2,
71-
152 => 1,
72-
155 => 2,
71+
155 => 1,
7372
159 => 1,
7473
166 => 1,
7574
173 => 1,
@@ -123,7 +122,6 @@ public function getErrorList($testFile)
123122
792 => 1,
124123
794 => 1,
125124
797 => 1,
126-
801 => 1,
127125
828 => 1,
128126
840 => 1,
129127
852 => 1,

0 commit comments

Comments
 (0)