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.

Sign up for GitHub

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

Jump to bottom

PHP 8.2 | Squiz/VariableComment: allow for stand-alone null/true/false types and more #211

Merged
merged 2 commits into from
Jan 10, 2024
Merged

PHP 8.2 | Squiz/VariableComment: allow for stand-alone null/true/false types and more #211

merged 2 commits into from
Jan 10, 2024

Commits on Jan 10, 2024

  1. PHP 8.2 | Squiz/VariableComment: allow for stand-alone null/true/fals… 

    …e types

Since PHP 8.2, stand-alone use of `true`, `false` and `null` is allowed.

However, if these were used as the type for a property, the docblock would not be found.

Fixed now.

Includes unit tests.
    @jrfnl
    jrfnl committed Jan 10, 2024
    Configuration menu
    Copy the full SHA
    7725dec View commit details
    Browse the repository at this point in the history

  2. Squiz/VariableComment: allow for various other missing types 

    There were some more allowed types missing from the list of tokens to skip over to find a docblock.

Fixed now.

Includes unit tests.
    @jrfnl
    jrfnl committed Jan 10, 2024
    Configuration menu
    Copy the full SHA
    b04f0a6 View commit details
    Browse the repository at this point in the history