Skip to content

[php82] Suppress PHP 8.2+ warnings for dynamic properties.#688

Merged
swashata merged 1 commit intodevelopfrom
feature/suppress-missing-dynamic-properties
Mar 31, 2024
Merged

[php82] Suppress PHP 8.2+ warnings for dynamic properties.#688
swashata merged 1 commit intodevelopfrom
feature/suppress-missing-dynamic-properties

Conversation

@bahiirwa
Copy link
Contributor

@bahiirwa bahiirwa commented Feb 11, 2024

Reference: [https://wiki.php.net/rfc/deprecate_dynamic_properties PHP RFC: Deprecate dynamic properties].

How to Test.

  1. Ensure your WP instance is running on PHP 8.2+
  2. Install Query Monitor with the latest SDK develop or master branch in use with any plugin.
    CleanShot 2024-02-11 at 18 07 00
  3. Change to the feature/suppress-missing-dynamic-properties branch
    CleanShot 2024-02-11 at 17 51 06

Copy link
Contributor

@swashata swashata left a comment

Choose a reason for hiding this comment

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

LGTM

@swashata swashata merged commit 13bbba6 into develop Mar 31, 2024
@swashata swashata deleted the feature/suppress-missing-dynamic-properties branch March 31, 2024 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants