Skip to content
Navigation Menu
Sign in
Appearance settings
Platform
AI CODE CREATION
GitHub Copilot
Write better code with AI
GitHub Copilot app
Direct agents from issue to merge
MCP Registry
Integrate external tools
DEVELOPER WORKFLOWS
Actions
Automate any workflow
Codespaces
Instant dev environments
Issues
Plan and track work
Code Review
Manage code changes
Code Quality
Enforce quality at merge
APPLICATION SECURITY
GitHub Advanced Security
Find and fix vulnerabilities
Code security
Secure your code as you build
Secret protection
Stop leaks before they start
EXPLORE
Why GitHub
Documentation
Blog
Changelog
Marketplace
View all features
Solutions
BY COMPANY SIZE
Enterprises
Small and medium teams
Startups
Nonprofits
BY USE CASE
App Modernization
DevSecOps
DevOps
CI/CD
View all use cases
BY INDUSTRY
Healthcare
Financial services
Manufacturing
Government
View all industries
View all solutions
Resources
EXPLORE BY TOPIC
AI
Software Development
DevOps
Security
View all topics
EXPLORE BY TYPE
Customer stories
Events & webinars
Ebooks & reports
Business insights
GitHub Skills
SUPPORT & SERVICES
Documentation
Customer support
Community forum
Trust center
Partners
View all resources
Open Source
COMMUNITY
GitHub Sponsors
Fund open source developers
PROGRAMS
Security Lab
Maintainer Community
GitHub Stars
Archive Program
REPOSITORIES
Topics
Trending
Collections
Enterprise
ENTERPRISE SOLUTIONS
Enterprise platform
AI-powered developer platform
AVAILABLE ADD-ONS
GitHub Advanced Security
Enterprise-grade security features
Copilot for Business
Enterprise-grade AI features
Premium Support
Enterprise-grade 24/7 support
Pricing
Search
/
Sign in
Sign up
Appearance settings
You signed in with another tab or window.
Reload
to refresh your session.
You signed out in another tab or window.
Reload
to refresh your session.
You switched accounts on another tab or window.
Reload
to refresh your session.
Dismiss alert
{{ message }}
stellarwp
/
coding-standards
Public
Notifications
You must be signed in to change notification settings
Fork
1
Star
8
Code
Pull requests
1
Actions
Security and quality
0
Insights
Additional navigation options
Code
Pull requests
Actions
Security and quality
Insights
feat: hook handler argument types (sniff + PHPStan rule)
- #28
#28
Draft
d4mation
wants to merge 17 commits into
main
stellarwp/coding-standards:main
from
feature/hook-handler-types
stellarwp/coding-standards:feature/hook-handler-types
Copy head branch name to clipboard
Conversation
Commits
17
(17)
Checks
Files changed
Draft
feat: hook handler argument types (sniff + PHPStan rule)
#28
d4mation
wants to merge 17 commits into
main
stellarwp/coding-standards:main
from
feature/hook-handler-types
stellarwp/coding-standards:feature/hook-handler-types
Copy head branch name to clipboard
Commits
Commits on Jul 23, 2026
feat: add hook handler argument types checks
Show description for ec4104c
d4mation
committed
ec4104c
View commit details
Copy full SHA for ec4104c
Browse repository at this point
Commits on Jul 24, 2026
fix: enforce type-less filter handlers; resolve &$this callbacks
Show description for b9f414a
d4mation
committed
b9f414a
View commit details
Copy full SHA for b9f414a
Browse repository at this point
refactor: drop prefix config; forbid native param types on all handlers
Show description for a340807
d4mation
committed
a340807
View commit details
Copy full SHA for a340807
Browse repository at this point
docs: advise validating/casting hook arguments inside the callback
Show description for 99d47e8
d4mation
committed
99d47e8
View commit details
Copy full SHA for 99d47e8
Browse repository at this point
feat: resolve container callbacks in the PHPStan rule
Show description for cc7bd30
d4mation
committed
cc7bd30
View commit details
Copy full SHA for cc7bd30
Browse repository at this point
docs: note container-callback support in the capability table
d4mation
committed
47b4c85
View commit details
Copy full SHA for 47b4c85
Browse repository at this point
feat: resolve hook-registration wrapper methods in the PHPStan rule
Show description for 4f51487
d4mation
committed
4f51487
View commit details
Copy full SHA for 4f51487
Browse repository at this point
feat: resolve $this->add_action/add_filter wrappers in the sniff
Show description for 6cbe578
d4mation
committed
6cbe578
View commit details
Copy full SHA for 6cbe578
Browse repository at this point
ci: add GitHub Actions workflow running composer test across PHP 7.4-8.4
d4mation
committed
ed358ca
View commit details
Copy full SHA for ed358ca
Browse repository at this point
ci: allow manual workflow_dispatch runs
d4mation
committed
0d990ed
View commit details
Copy full SHA for 0d990ed
Browse repository at this point
ci: run PHPUnit suite directly with --testdox; drop workflow_dispatch
d4mation
committed
6cd789a
View commit details
Copy full SHA for 6cd789a
Browse repository at this point
ci: bump checkout to v7 and composer-install to v4 (Node 24)
d4mation
committed
2268a84
View commit details
Copy full SHA for 2268a84
Browse repository at this point
docs: note PHPStan 2.x (php-parser 5.x) would remove the 7.4 test skip
Show description for 0d28d13
d4mation
committed
0d28d13
View commit details
Copy full SHA for 0d28d13
Browse repository at this point
fix: report but do not auto-fix wrapper-resolved hook handlers
Show description for b04547e
d4mation
committed
b04547e
View commit details
Copy full SHA for b04547e
Browse repository at this point
docs: complete param/return docblocks and scope the phpcs suppression
Show description for 28b21f5
d4mation
committed
28b21f5
View commit details
Copy full SHA for 28b21f5
Browse repository at this point
refactor: dedupe hook-handler resolution and report each hook name
Show description for 99a94af
d4mation
committed
99a94af
View commit details
Copy full SHA for 99a94af
Browse repository at this point
test: avoid stdlib call in multi-hook fixture (fixes PHP 8.0 CI)
Show description for d1d674f
d4mation
committed
d1d674f
View commit details
Copy full SHA for d1d674f
Browse repository at this point
You can’t perform that action at this time.