Skip to content

Make phpdoc comments accurate#261

Merged
Majkl578 merged 1 commit into
doctrine:masterfrom
greg0ire:fix-phpstan
Mar 29, 2019
Merged

Make phpdoc comments accurate#261
Majkl578 merged 1 commit into
doctrine:masterfrom
greg0ire:fix-phpstan

Conversation

@greg0ire

Copy link
Copy Markdown
Member

This fixes the phpstan job of the build.
Refs #255

Comment thread lib/Doctrine/Annotations/Metadata/Builder/PropertyMetadataBuilder.php Outdated
@adaamz

adaamz commented Mar 25, 2019

Copy link
Copy Markdown

I think string is not acceptable for $enum.

@greg0ire

Copy link
Copy Markdown
Member Author

Me too :P

@adaamz adaamz left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

lgtm

@adaamz adaamz left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I just noticed that in PropertyMetadata is @param for property, there should be @var, please fix.

Comment thread lib/Doctrine/Annotations/Metadata/PropertyMetadata.php Outdated
adaamz
adaamz previously approved these changes Mar 26, 2019

@Majkl578 Majkl578 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks ok, besides the ? part. :)

Comment thread lib/Doctrine/Annotations/Metadata/Builder/PropertyMetadataBuilder.php Outdated
Comment thread lib/Doctrine/Annotations/Metadata/PropertyMetadata.php Outdated
Comment thread lib/Doctrine/Annotations/Metadata/PropertyMetadata.php Outdated
Comment thread lib/Doctrine/Annotations/Metadata/PropertyMetadata.php Outdated
Comment thread lib/Doctrine/Annotations/Metadata/PropertyMetadata.php Outdated
Comment thread lib/Doctrine/Annotations/Metadata/Builder/PropertyMetadataBuilder.php Outdated
This fixes the phpstan job of the build.
Refs doctrine#255
@Majkl578 Majkl578 merged commit ecccd57 into doctrine:master Mar 29, 2019
@Majkl578 Majkl578 self-assigned this Mar 29, 2019
@Majkl578 Majkl578 added this to the v2.0.0 milestone Mar 29, 2019
@Majkl578

Copy link
Copy Markdown
Contributor

👍

@greg0ire greg0ire deleted the fix-phpstan branch March 29, 2019 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants