[PR #70] [MERGED] Add workflow job for phpstan #95

New issue

Closed

opened 2026-05-07 00:18:03 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-07 00:18:03 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/plugins/pull/70
Author: @Boy132
Created: 1/1/2026
Status: ✅ Merged
Merged: 1/1/2026
Merged by: @Boy132

Base: main ← Head: phpstan-workflow

📝 Commits (7)

187dcb6 add workflow job for phpstan
aaf375b use custom phpstan.neon
84d741a fix name
cbfe871 add ignoreErrors
394daa3 fix some errors
b07b18c fix remaining errors
b1c97c0 small cleanup

📊 Changes

17 files changed (+113 additions, -50 deletions)

View changed files

➕ .github/workflows/lint.yml (+74 -0)
➖ .github/workflows/pint.yml (+0 -29)
📝 billing/src/Filament/App/Widgets/ProductWidget.php (+1 -1)
📝 billing/src/Models/Coupon.php (+2 -2)
📝 billing/src/Models/Order.php (+6 -6)
📝 billing/src/Models/Product.php (+2 -2)
📝 billing/src/Models/ProductPrice.php (+1 -1)
📝 pastefox-share/src/Filament/Components/Actions/UploadLogsAction.php (+1 -1)
➕ phpstan.neon (+13 -0)
📝 player-counter/database/Seeders/PlayerCounterSeeder.php (+1 -0)
📝 player-counter/src/Filament/Server/Pages/PlayersPage.php (+3 -3)
📝 player-counter/src/Filament/Server/Widgets/ServerPlayerWidget.php (+2 -2)
📝 subdomains/src/Filament/Server/Resources/Subdomains/SubdomainResource.php (+1 -1)
📝 subdomains/src/Models/CloudflareDomain.php (+1 -0)
📝 subdomains/src/Models/Subdomain.php (+3 -0)
📝 user-creatable-servers/src/Filament/Server/Pages/ServerResourcePage.php (+1 -1)
📝 user-creatable-servers/src/Models/UserResourceLimits.php (+1 -1)

📄 Description

Summary by CodeRabbit

Chores
- Consolidated CI linting into a single workflow for consistent checks and added a new static-analysis job.
- Removed the legacy Pint workflow.
- Added static analysis configuration to improve code validation.
- Cleaned up internal annotations and lint suppression comments across various modules to streamline tooling feedback.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/pelican-dev/plugins/pull/70 **Author:** [@Boy132](https://github.com/Boy132) **Created:** 1/1/2026 **Status:** ✅ Merged **Merged:** 1/1/2026 **Merged by:** [@Boy132](https://github.com/Boy132) **Base:** `main` ← **Head:** `phpstan-workflow` --- ### 📝 Commits (7) - [`187dcb6`](https://github.com/pelican-dev/plugins/commit/187dcb6aeb57225f6788950c0630b412da635183) add workflow job for phpstan - [`aaf375b`](https://github.com/pelican-dev/plugins/commit/aaf375bde920fb27742821af3f0f650dec3f147f) use custom phpstan.neon - [`84d741a`](https://github.com/pelican-dev/plugins/commit/84d741ab0120ed365fb159e6a755bc42a316537c) fix name - [`cbfe871`](https://github.com/pelican-dev/plugins/commit/cbfe871f1b748ba778415fb1164edebd1bb1fc54) add ignoreErrors - [`394daa3`](https://github.com/pelican-dev/plugins/commit/394daa363219762804f23f08df3a596c95005d6a) fix some errors - [`b07b18c`](https://github.com/pelican-dev/plugins/commit/b07b18cfd45202777a505ec0ebee3e5775c37f80) fix remaining errors - [`b1c97c0`](https://github.com/pelican-dev/plugins/commit/b1c97c06b5d1d00fc4b74adbe393f49eba9073d9) small cleanup ### 📊 Changes **17 files changed** (+113 additions, -50 deletions) <details> <summary>View changed files</summary> ➕ `.github/workflows/lint.yml` (+74 -0) ➖ `.github/workflows/pint.yml` (+0 -29) 📝 `billing/src/Filament/App/Widgets/ProductWidget.php` (+1 -1) 📝 `billing/src/Models/Coupon.php` (+2 -2) 📝 `billing/src/Models/Order.php` (+6 -6) 📝 `billing/src/Models/Product.php` (+2 -2) 📝 `billing/src/Models/ProductPrice.php` (+1 -1) 📝 `pastefox-share/src/Filament/Components/Actions/UploadLogsAction.php` (+1 -1) ➕ `phpstan.neon` (+13 -0) 📝 `player-counter/database/Seeders/PlayerCounterSeeder.php` (+1 -0) 📝 `player-counter/src/Filament/Server/Pages/PlayersPage.php` (+3 -3) 📝 `player-counter/src/Filament/Server/Widgets/ServerPlayerWidget.php` (+2 -2) 📝 `subdomains/src/Filament/Server/Resources/Subdomains/SubdomainResource.php` (+1 -1) 📝 `subdomains/src/Models/CloudflareDomain.php` (+1 -0) 📝 `subdomains/src/Models/Subdomain.php` (+3 -0) 📝 `user-creatable-servers/src/Filament/Server/Pages/ServerResourcePage.php` (+1 -1) 📝 `user-creatable-servers/src/Models/UserResourceLimits.php` (+1 -1) </details> ### 📄 Description  ## Summary by CodeRabbit * **Chores** * Consolidated CI linting into a single workflow for consistent checks and added a new static-analysis job. * Removed the legacy Pint workflow. * Added static analysis configuration to improve code validation. * Cleaned up internal annotations and lint suppression comments across various modules to streamline tooling feedback. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub>  --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

BreizhHardware

2026-05-07 00:18:03 +02:00

closed this issue
added the
pull-request
label

No labels

pull-request

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/plugins#95

No description provided.

Rows
Columns