Skip to content

Conversation

@layoutd
Copy link
Contributor

@layoutd layoutd commented Oct 30, 2025

All Submissions:

  • Have you followed the Contributing guidelines?
  • Does your code follow the WordPress' coding standards?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes proposed in this Pull Request:

Updates all Composer dependencies to their latest versions, with a focus on adding PHP 8.4/8.5 support while maintaining PHP 7.4 compatibility.

Key updates:

  • PHP_CodeSniffer: 3.7.2 => 3.13.4 (adds PHP 8.4+ support)
  • WordPress Coding Standards: 2.3.0 => 3.2.0
  • WooCommerce Sniffs: 0.1.3 => 1.0.1
  • Various supporting dependencies updated

This resolves deprecation warnings and errors when running PHPCS on PHP 8.4+.

How to test the changes in this Pull Request:

  1. Check out this branch
  2. Run composer install
  3. Run vendor/bin/phpcs --version to verify PHPCS 3.13.4
  4. Run vendor/bin/phpcs includes/Generator/Customer.php to verify it works without PHP 8.4 compatibility errors
  5. Verify no regressions in existing functionality

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully run tests with your changes locally?

Changelog entry

Update Composer dependencies to support PHP 8.4 and 8.5

@layoutd layoutd requested a review from Copilot October 30, 2025 00:29
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@layoutd layoutd self-assigned this Oct 30, 2025
@layoutd layoutd merged commit cb8a49d into trunk Oct 30, 2025
@layoutd layoutd deleted the update-composer-php85 branch October 30, 2025 00:30
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