Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: add phpstan static analysis minimum #4511

Merged
merged 71 commits into from
Nov 28, 2022
Merged

chore: add phpstan static analysis minimum #4511

merged 71 commits into from
Nov 28, 2022

Conversation

lancepioch
Copy link
Contributor

No description provided.

@lancepioch lancepioch marked this pull request as ready for review November 1, 2022 18:47
@lancepioch lancepioch changed the title Add Static Analysis minimun with phpstan Add Static Analysis minimum with phpstan Nov 6, 2022
@matthewpi
Copy link
Member

Let's get some of these conflicts fixed up and get this merged in ASAP, it should help us when we go to make additional changes.

lancepioch and others added 10 commits November 27, 2022 22:51
# Conflicts:
#	app/Http/Controllers/Admin/Servers/ServerTransferController.php
#	app/Models/Node.php
#	app/Repositories/Eloquent/NodeRepository.php
#	app/Services/Deployment/FindViableNodesService.php
@matthewpi matthewpi changed the title Add Static Analysis minimum with phpstan chore: add phpstan static analysis minimum Nov 28, 2022
@matthewpi matthewpi merged commit a1a5275 into develop Nov 28, 2022
@matthewpi matthewpi deleted the analysis branch November 28, 2022 16:56
colecrouter pushed a commit to colecrouter/pterodactyl-panel that referenced this pull request Apr 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🧨 Awaiting release
Development

Successfully merging this pull request may close these issues.

2 participants