Skip to content

Update PHPStan analysis to match dxpr_theme_helper approach#13

Merged
jjroelofs merged 1 commit into1.xfrom
fix-batch-operation-2
Sep 10, 2025
Merged

Update PHPStan analysis to match dxpr_theme_helper approach#13
jjroelofs merged 1 commit into1.xfrom
fix-batch-operation-2

Conversation

@jjroelofs
Copy link
Copy Markdown
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)
  • Fix module symlink path to analyze_ai_sentiments (was incorrectly analyze_ai_brand_voice)
  • Maintain Drupal 11 compatibility with statistics module requirement

Test plan

  • Verify PHPStan analysis runs with new configuration
  • Confirm code quality standards compliance
  • Test Docker compose integration

This brings the analyze_ai_sentiments 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
- Fix module symlink path to analyze_ai_sentiments (was incorrectly analyze_ai_brand_voice)
- 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 423ab79 into 1.x Sep 10, 2025
0 of 2 checks passed
@jjroelofs jjroelofs deleted the fix-batch-operation-2 branch September 10, 2025 14:54
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.

1 participant