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.

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

Add @extends tag to Doctrine repository template. #1088

Conversation

hhamon
Copy link

@hhamon hhamon commented Apr 3, 2022

Adding the @extends phpdoc tag on generated Doctrine repository classes will prevent phpstan from generating an error when it's installed in the project.

@hhamon hhamon force-pushed the feature/generate-extends-phpdoc-tag-doctrine-entity branch from 918e3d7 to d2ba22b Compare April 3, 2022 15:12
@hhamon hhamon force-pushed the feature/generate-extends-phpdoc-tag-doctrine-entity branch from d2ba22b to 73b5aee Compare April 3, 2022 15:12
@hhamon
Copy link
Author

hhamon commented Apr 3, 2022

@weaverryan @kbond ready for review :)

@weaverryan
Copy link
Member

Thanks Hugo!

@weaverryan weaverryan merged commit b2eacf5 into symfony:main Apr 5, 2022
@jrushlow jrushlow added the Feature New Feature label Apr 5, 2022
@hhamon hhamon deleted the feature/generate-extends-phpdoc-tag-doctrine-entity branch April 5, 2022 19:32
@jrushlow jrushlow mentioned this pull request Apr 21, 2022
weaverryan added a commit that referenced this pull request Jul 10, 2023
…y template (gremo)

This PR was merged into the 1.0-dev branch.

Discussion
----------

Add conditional `@implements` tag to Doctrine repository template

Adding the `@implements` phpdoc tag on generated Doctrine repository classes will help IDE and prevent tools like phpstan from generating errors.

`@implements` is only added if `$with_password_upgrade` is set to true, avoiding useless phpdocs.

![Immagine 2023-07-06 120413](https://github.com/symfony/maker-bundle/assets/1532616/9e9c1b08-4f60-4a8b-862d-0f32cff2920b)

Similar to #1088

Commits
-------

dd3208f Add conditional `@implements` tag to Doctrine repository template
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants