Skip to content

Commit 299b448

Browse files
committed
Add PHP 8.2
1 parent d3044c1 commit 299b448

File tree

2 files changed

+40
-10
lines changed

2 files changed

+40
-10
lines changed

.github/workflows/test.yml

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,38 +6,49 @@ on:
66
pull_request:
77
branches: [ master ]
88

9+
permissions:
10+
contents: read
11+
912
jobs:
1013
test:
1114
name: PHP ${{ matrix.php-versions }} Test ${{ matrix.composer-flags }}
1215
runs-on: ubuntu-latest
1316
strategy:
1417
matrix:
15-
php-versions: ['7.4', '8.0', '8.1']
18+
php-versions: ['7.4', '8.0', '8.1', '8.2']
1619
composer-flags: ['', '--prefer-lowest']
1720
steps:
1821
- uses: actions/checkout@v2
22+
1923
- name: Setup PHP
2024
uses: shivammathur/setup-php@v2
2125
with:
2226
php-version: ${{ matrix.php-versions }}
2327
coverage: none
28+
2429
- name: Validate composer.json and composer.lock
2530
run: composer validate
31+
2632
- name: Install dependencies
2733
run: composer update --prefer-dist --no-progress --no-suggest --prefer-stable ${{ matrix.composer-flags }}
34+
2835
- name: Run test suite
2936
run: composer test
37+
3038
quality:
3139
name: Quality checks
3240
runs-on: ubuntu-latest
3341
steps:
3442
- uses: actions/checkout@v2
43+
3544
- name: Setup PHP
3645
uses: shivammathur/setup-php@v2
3746
with:
3847
php-version: 8.0
3948
coverage: none
49+
4050
- name: Install dependencies
4151
run: composer install --prefer-dist --no-progress --no-suggest
52+
4253
- name: run checks
4354
run: composer check

phpmd.xml.dist

Lines changed: 28 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -8,15 +8,6 @@
88
The Code Size Ruleset contains a collection of rules that find code size related problems.
99
</description>
1010

11-
<rule ref="rulesets/codesize.xml">
12-
<exclude name="TooManyPublicMethods"/>
13-
</rule>
14-
<rule ref="rulesets/codesize.xml/TooManyPublicMethods">
15-
<properties>
16-
<property name="ignorepattern" value="(^(set|get|test))i"/>
17-
</properties>
18-
</rule>
19-
2011
<rule ref="rulesets/controversial.xml"/>
2112
<rule ref="rulesets/design.xml">
2213
<exclude name="CouplingBetweenObjects" />
@@ -27,12 +18,40 @@
2718
</properties>
2819
</rule>
2920
<rule ref="rulesets/unusedcode.xml"/>
21+
22+
<!-- Naming rules -->
3023
<rule ref="rulesets/naming.xml">
24+
<exclude name="ShortVariable"/>
3125
<exclude name="LongVariable"/>
26+
<exclude name="LongClassName"/>
27+
</rule>
28+
<rule ref="rulesets/naming.xml/ShortVariable">
29+
<properties>
30+
<property name="exceptions" value="id,e,i,x,y,z" />
31+
</properties>
3232
</rule>
3333
<rule ref="rulesets/naming.xml/LongVariable">
3434
<properties>
3535
<property name="maximum" value="25"/>
36+
<property name="subtract-suffixes"
37+
value="Repository,Controller,Service,Hydrator,Provider,Factory,Builder,Manager,Handler,View,ViewModel,Repositories,Controllers,Services,Hydrators,Providers,Factories,Builders,Managers,Handlers,Views,ViewModels"/>
38+
</properties>
39+
</rule>
40+
41+
<rule ref="rulesets/codesize.xml">
42+
<exclude name="TooManyMethods"/>
43+
<exclude name="TooManyPublicMethods"/>
44+
</rule>
45+
<!-- exclude `function test*` from count -->
46+
<rule ref="rulesets/codesize.xml/TooManyMethods">
47+
<properties>
48+
<property name="ignorepattern" value="(^(set|get|test))"/>
49+
</properties>
50+
</rule>
51+
<!-- exclude `function test*` from count -->
52+
<rule ref="rulesets/codesize.xml/TooManyPublicMethods">
53+
<properties>
54+
<property name="ignorepattern" value="(^(set|get|test))"/>
3655
</properties>
3756
</rule>
3857
</ruleset>

0 commit comments

Comments
 (0)