Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve PHP 8.x linting #3

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

jeremyfelt
Copy link

The PHPCompatibilityWP package is tied to the 9.x release of the PHPCompatibility package.

The current 9.3.5 release of PHPCompatibility has not been updated since December 2019 and I believe is missing quite a few updated sniffs for PHP 8.x syntax.

See: PHPCompatibility/PHPCompatibility@9.3.5...develop

This change labels the PHPCompatibility develop branch as a 9.99.99 release, which satisifies the PHPCompatibilityWP requirements and improves scanning for PHP 8.x compatibility. This is the method recommended on the PHPCompatibilityWP repository.

Changes:

  • composer config minimum-stability dev
  • composer require --dev phpcompatibility/php-compatibility:"dev-develop as 9.99.99"
  • composer update phpcompatibility/phpcompatibility-wp

The PHPCompatibilityWP package is tied to the 9.x release of the
PHPCompatibility package.

The current 9.3.5 release of PHPCompatibility has not been updated
since December 2019 and I believe is missing quite a few updated
sniffs for PHP 8.x syntax.

See: PHPCompatibility/PHPCompatibility@9.3.5...develop

This labels the PHPCompatibility `develop` branch as a 9.99.99
release, which satisifies the PHPCompatibilityWP requirements and
improves scanning for PHP 8.x compatibility.

See: PHPCompatibility/PHPCompatibilityWP#40 (comment)

Changes:
* `composer config minimum-stability dev`
* `composer require --dev
  phpcompatibility/php-compatibility:"dev-develop as 9.99.99"`
* `composer update phpcompatibility/phpcompatibility-wp`
@jeremyfelt
Copy link
Author

There's a chance composer config minimum-stability dev is not necessary, though I'm not sure it impacts too much in this configuration.

@ahegyes
Copy link
Contributor

ahegyes commented Dec 12, 2024

The project is still actively worked on although there really hasn't been a new release in 5 years. Is this being solved some other way or is this still an issue ~1.5 years later? 😅

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