[PR #639] [MERGED] Enforce return and parameter types #866

Closed
opened 2026-02-04 18:57:07 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/639
Author: @lancepioch
Created: 10/20/2024
Status: Merged
Merged: 10/20/2024
Merged by: @lancepioch

Base: mainHead: lance/phpstan-return-types


📝 Commits (5)

  • c7a307a Enforce return and parameter types
  • 38be89a Pint
  • bb5955c Have to make this match the trait
  • 81c75f7 Merge branch 'main' into lance/phpstan-return-types
  • e162374 Add return types

📊 Changes

61 files changed (+145 additions, -125 deletions)

View changed files

📝 app/Console/Commands/Environment/EmailSettingsCommand.php (+4 -4)
📝 app/Console/Commands/Schedule/ProcessRunnableCommand.php (+1 -1)
📝 app/Console/Commands/UpgradeCommand.php (+1 -1)
📝 app/Exceptions/DisplayException.php (+7 -5)
📝 app/Extensions/Themes/Theme.php (+3 -3)
📝 app/Filament/Pages/Installer/PanelInstaller.php (+4 -3)
📝 app/Filament/Pages/Installer/Steps/DatabaseStep.php (+1 -1)
📝 app/Filament/Pages/Installer/Steps/RedisStep.php (+1 -1)
📝 app/Filament/Resources/ApiKeyResource.php (+2 -1)
📝 app/Filament/Resources/DatabaseHostResource/Pages/CreateDatabaseHost.php (+4 -2)
📝 app/Filament/Resources/DatabaseHostResource/Pages/EditDatabaseHost.php (+10 -4)
📝 app/Filament/Resources/DatabaseHostResource/RelationManagers/DatabasesRelationManager.php (+2 -1)
📝 app/Filament/Resources/NodeResource/Pages/EditNode.php (+2 -2)
📝 app/Filament/Resources/ServerResource/Pages/CreateServer.php (+1 -1)
📝 app/Filament/Resources/ServerResource/Pages/EditServer.php (+1 -1)
📝 app/Filament/Resources/UserResource/Pages/EditProfile.php (+8 -2)
📝 app/Http/Controllers/Admin/ServersController.php (+2 -4)
📝 app/Http/Controllers/Api/Application/ApplicationApiController.php (+1 -1)
📝 app/Http/Controllers/Api/Remote/ActivityProcessingController.php (+1 -1)
📝 app/Http/Controllers/Auth/AbstractLoginController.php (+2 -2)

...and 41 more files

📄 Description

No description provided


🔄 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/panel/pull/639 **Author:** [@lancepioch](https://github.com/lancepioch) **Created:** 10/20/2024 **Status:** ✅ Merged **Merged:** 10/20/2024 **Merged by:** [@lancepioch](https://github.com/lancepioch) **Base:** `main` ← **Head:** `lance/phpstan-return-types` --- ### 📝 Commits (5) - [`c7a307a`](https://github.com/pelican-dev/panel/commit/c7a307af6eab55e1ab81e9abef2cae9658220c3c) Enforce return and parameter types - [`38be89a`](https://github.com/pelican-dev/panel/commit/38be89a71e6b0251a34d536ea4bbaec025171dcd) Pint - [`bb5955c`](https://github.com/pelican-dev/panel/commit/bb5955cff421d83bb5cdf3ff832f88c6a2ba3428) Have to make this match the trait - [`81c75f7`](https://github.com/pelican-dev/panel/commit/81c75f79666883bd13be980413bc3dceacf6d714) Merge branch 'main' into lance/phpstan-return-types - [`e162374`](https://github.com/pelican-dev/panel/commit/e162374e156d5e77499aeeddcdaed0d73853edf1) Add return types ### 📊 Changes **61 files changed** (+145 additions, -125 deletions) <details> <summary>View changed files</summary> 📝 `app/Console/Commands/Environment/EmailSettingsCommand.php` (+4 -4) 📝 `app/Console/Commands/Schedule/ProcessRunnableCommand.php` (+1 -1) 📝 `app/Console/Commands/UpgradeCommand.php` (+1 -1) 📝 `app/Exceptions/DisplayException.php` (+7 -5) 📝 `app/Extensions/Themes/Theme.php` (+3 -3) 📝 `app/Filament/Pages/Installer/PanelInstaller.php` (+4 -3) 📝 `app/Filament/Pages/Installer/Steps/DatabaseStep.php` (+1 -1) 📝 `app/Filament/Pages/Installer/Steps/RedisStep.php` (+1 -1) 📝 `app/Filament/Resources/ApiKeyResource.php` (+2 -1) 📝 `app/Filament/Resources/DatabaseHostResource/Pages/CreateDatabaseHost.php` (+4 -2) 📝 `app/Filament/Resources/DatabaseHostResource/Pages/EditDatabaseHost.php` (+10 -4) 📝 `app/Filament/Resources/DatabaseHostResource/RelationManagers/DatabasesRelationManager.php` (+2 -1) 📝 `app/Filament/Resources/NodeResource/Pages/EditNode.php` (+2 -2) 📝 `app/Filament/Resources/ServerResource/Pages/CreateServer.php` (+1 -1) 📝 `app/Filament/Resources/ServerResource/Pages/EditServer.php` (+1 -1) 📝 `app/Filament/Resources/UserResource/Pages/EditProfile.php` (+8 -2) 📝 `app/Http/Controllers/Admin/ServersController.php` (+2 -4) 📝 `app/Http/Controllers/Api/Application/ApplicationApiController.php` (+1 -1) 📝 `app/Http/Controllers/Api/Remote/ActivityProcessingController.php` (+1 -1) 📝 `app/Http/Controllers/Auth/AbstractLoginController.php` (+2 -2) _...and 41 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-04 18:57:07 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel#866