pFad - Phone/Frame/Anonymizer/Declutterfier! Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

URL: http://github.com/phpstan/phpstan/pull/14850

pull-requests-be6017ec12798e73.css" /> Update Error Docs by phpstan-bot · Pull Request #14850 · phpstan/phpstan · GitHub
Skip to content

Update Error Docs - #14850

Open
phpstan-bot wants to merge 1 commit into
phpstan:2.2.xfrom
phpstan-bot:create-pull-request/patch-l22375c
Open

Update Error Docs#14850
phpstan-bot wants to merge 1 commit into
phpstan:2.2.xfrom
phpstan-bot:create-pull-request/patch-l22375c

Conversation

@phpstan-bot

Copy link
Copy Markdown
Contributor

The latest website/src/errorsIdentifiers.json update shifted the source locations for the phpunit.attributeRequiresPhpVersion identifier (from 4 to 5 locations), reflecting a new error message added to AttributeRequiresPhpVersionRule in phpstan-phpunit.

After checking the updated rule source and test fixtures, the existing documentation only described the "missing operator" case, while the rule actually reports several distinct problems under one identifier. The docs were updated to cover all of them:

  • Version requirement is missing operator — bare version like '8.1' with no comparison operator.
  • Version requirement without operator is deprecated — same bare form when phpstan-deprecation-rules is installed.
  • Version requirement is incomplete — the new message; version not in full major.minor.patch form (PHPUnit 12.5+, bleeding edge).
  • Version requirement will always evaluate to false — constraint that can never match any analysed PHP version.
  • Unparsable constraint strings (e.g. 'abc').

The shortDescription frontmatter was broadened to reflect the wider scope, and a fix example for writing versions in full major.minor.patch form was added.

No new identifiers required documentation — all 1097 identifiers in the JSON already have corresponding .md files.

- AttributeRequiresPhpVersionRule gained a new error message
  ("Version requirement is incomplete.") and shifted source lines
  in errorsIdentifiers.json
- Expanded the doc to cover all conditions reported by the
  identifier: missing operator, deprecated bare syntax, incomplete
  version, always-false constraints, and unparsable constraints
- Added a fix example for writing versions in full major.minor.patch form
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.

1 participant

pFad - Phonifier reborn

Pfad - The Proxy pFad © 2024 Your Company Name. All rights reserved.





Check this box to remove all script contents from the fetched content.



Check this box to remove all images from the fetched content.


Check this box to remove all CSS styles from the fetched content.


Check this box to keep images inefficiently compressed and original size.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy