Skip to content

Conversation

@jjroelofs
Copy link
Contributor

Summary

  • Switch from mglaman/drupal-check to direct PHPStan usage for consistency with dxpr_theme_helper
  • Add inline phpstan.neon configuration with level 5 analysis
  • Use same PHPStan extensions (phpstan/phpstan, mglaman/phpstan-drupal, phpstan/phpstan-deprecation-rules)
  • Maintain Drupal 11 compatibility with statistics module requirement

Test plan

  • Verify PHPStan analysis runs successfully with new configuration
  • Confirm 9 issues detected at level 5 (showing proper analysis depth)
  • Test Docker compose integration works correctly
  • Validate coding standards compliance

This brings the analyze module's static analysis approach in line with dxpr_theme_helper for better maintainability across DXPR modules.

🤖 Generated with Claude Code

- Switch from mglaman/drupal-check to direct PHPStan usage
- Add inline phpstan.neon configuration with level 5 analysis
- Use same PHPStan extensions as dxpr_theme_helper for consistency
- Maintain Drupal 11 compatibility with statistics module requirement

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@jjroelofs jjroelofs merged commit c82c8ad into 1.0.x Sep 10, 2025
1 of 2 checks passed
@jjroelofs jjroelofs deleted the fix/linting-issues-1.0.x branch September 10, 2025 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants