feat: add comprehensive Drupal linting and checking infrastructure#4
Open
feat: add comprehensive Drupal linting and checking infrastructure#4
Conversation
Add drupal-lint service with PHPCompatibility, Drupal, and DrupalPractice standards Add drupal-lint-auto-fix for automated code standard corrections Add drupal-check service with full dependency management for ai_social_posts Configure proper module-specific ignore patterns and dependency installation Ensure compatibility with Drupal 11.2.x-dev and maxlength module requirements 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add GitHub Actions workflow for automated Drupal linting and code quality checks - Update docker-compose.yml to use Drupal 11.x-dev consistently - Add automated PHPCompatibility checks for PHP 8.3+ compatibility - Configure workflows to run on push and pull requests to main/1.x branches 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Separate drupal-check into dedicated job for clearer status reporting - Maintain existing drupal-lint and compatibility checks - Improve workflow organization and readability 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Move scripts to scripts/linters/ directory following standard structure - Replace mglaman/drupal-check with PHPStan analysis for better static analysis - Add environment variable support for flexible Docker configurations - Update workflow to use --profile lint for cleaner service management - Implement proper dependency management with module-specific requirements - Align with dxpr_theme_helper standards for consistent tooling across projects 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
docker compose run --rm drupal-lintto verify linting worksdocker compose run --rm drupal-checkto verify dependency checking worksdocker compose run --rm drupal-lint-auto-fixto verify auto-fixing works🤖 Generated with Claude Code