Skip to content

⬆️ Support drupal/coder ^8 and ^9#94

Merged
kimpepper merged 5 commits intomainfrom
issue-93-coder-9-php-85
Apr 21, 2026
Merged

⬆️ Support drupal/coder ^8 and ^9#94
kimpepper merged 5 commits intomainfrom
issue-93-coder-9-php-85

Conversation

@kimpepper
Copy link
Copy Markdown
Member

@kimpepper kimpepper commented Apr 21, 2026

  • Support both drupal/coder ^8.3.26 and ^9.0
  • Support both squizlabs/php_codesniffer ^3.9.1 and ^4.0.1
  • Use feature detection for conditional test assertions
  • Test PHP 8.2–8.5 with lowest/highest deps

Closes #93

- Update drupal/coder ^8.3.26 → ^9.0
- Update squizlabs/php_codesniffer ^3.11.1 → ^4.0.1
- Update slevomat/coding-standard ^8.16.0 → ^8.28.1
- Update dev dependencies to pin minimum versions
- Update CI matrix to test PHP 8.2 and 8.5
- Fix renamed sniff reference in tests (Coder 9 API change)
- Update PHPCS ignore annotation syntax for v4

Closes #93
@kimpepper kimpepper requested a review from dpi April 21, 2026 00:34
@kimpepper
Copy link
Copy Markdown
Member Author

I created this after I ran into a PHP 8.5 issue on previousnext/bom-weather#16

Comment thread composer.json Outdated
Comment thread .github/workflows/workflows.yml Outdated
Comment thread composer.json Outdated
- Allow drupal/coder ^8.3.26 || ^9.0 and squizlabs/php_codesniffer ^3.9.1 || ^4.0.1
- Add version detection helper for conditional test assertions
- Test PHP 8.2-8.5 with lowest/highest dependency matrix
@kimpepper kimpepper changed the title ⬆️ Support drupal/coder ^9.0 for PHP 8.5 compatibility Support drupal/coder ^8 and ^9 for PHP 8.5 compatibility Apr 21, 2026
@kimpepper kimpepper requested a review from dpi April 21, 2026 01:43
Comment thread tests/Sniffs/DrupalRulesetTest.php
@kimpepper kimpepper changed the title Support drupal/coder ^8 and ^9 for PHP 8.5 compatibility ⬆️ Support drupal/coder ^8 and ^9 Apr 21, 2026
@kimpepper kimpepper merged commit 6744c07 into main Apr 21, 2026
16 checks passed
@kimpepper kimpepper deleted the issue-93-coder-9-php-85 branch April 21, 2026 04:20
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.

Support drupal/coder ^9.0 for PHP 8.5 compatibility

2 participants