Skip to content

Conversation

@oliverklee
Copy link
Collaborator

It has turned out that PHPStan does not need them.

Fixes #1396

It has turned out that PHPStan does not need them.

Fixes #1396
@coveralls
Copy link

Coverage Status

coverage: 59.658%. remained the same
when pulling 5ff5e41 on cleanup/constants-annotations
into 3f2ec3d on main.

Copy link
Collaborator

@JakeQZ JakeQZ left a comment

Choose a reason for hiding this comment

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

Makes sense, as it can see what type they are, and, being constants, they can't be changed.

@JakeQZ JakeQZ merged commit c29c15a into main Nov 27, 2025
23 checks passed
@JakeQZ JakeQZ deleted the cleanup/constants-annotations branch November 27, 2025 18:49
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.

Drop PHPDoc type annotations for constants

4 participants