Skip to content

Commit 6d13252

Browse files
committed
fix(ci): validate LibForm against PocketMine API
1 parent f3df67e commit 6d13252

13 files changed

Lines changed: 1082 additions & 24 deletions

File tree

.github/scripts/analyse.php

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,6 @@
1818
}
1919

2020
$phpstan = $root . '/vendor/bin/phpstan';
21-
if (PHP_OS_FAMILY === 'Windows') {
22-
$phpstan .= '.bat';
23-
}
2421

2522
if (!is_file($phpstan)) {
2623
echo "PHPStan is not installed; skipping analysis.\n";

.github/workflows/quality.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ jobs:
1414

1515
steps:
1616
- name: Checkout
17-
uses: actions/checkout@v4
17+
uses: actions/checkout@v5
1818

1919
- name: Setup PHP
2020
uses: shivammathur/setup-php@v2
@@ -32,7 +32,7 @@ jobs:
3232

3333
- name: Install Composer dependencies
3434
if: ${{ hashFiles('composer.json') != '' }}
35-
run: composer install --prefer-dist --no-interaction --no-progress
35+
run: composer install --prefer-dist --no-interaction --no-progress --ignore-platform-req='ext-*'
3636

3737
- name: Run Composer quality script
3838
if: ${{ hashFiles('composer.json') != '' }}

CHANGELOG.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,16 @@ The format is inspired by Keep a Changelog and uses semantic versioning where po
1313
- Quality workflow scaffolding.
1414
- Compatibility and standalone/embedded documentation.
1515

16+
### Changed
17+
18+
- Quality analysis now uses the real PocketMine-MP 5 API instead of unresolved framework types.
19+
- Form collections now declare their key and value types for static analysis.
20+
21+
### Fixed
22+
23+
- PHPStan now runs correctly through the Composer quality script on Windows.
24+
- Removed unused persisted constructor state while preserving the public form APIs.
25+
1626
## [2.0.0] - TBD
1727

1828
### Notes

composer.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,8 @@
1212
"php": "^8.2"
1313
},
1414
"require-dev": {
15-
"phpstan/phpstan": "^1.10"
15+
"phpstan/phpstan": "^1.10",
16+
"pocketmine/pocketmine-mp": "^5.0"
1617
},
1718
"scripts": {
1819
"lint": "@php .github/scripts/lint.php",

0 commit comments

Comments
 (0)