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

fix(metadata) Added phpdoc to ORM #6387

Merged
merged 1 commit into from
May 31, 2024
Merged

Conversation

stephanvierkant
Copy link
Contributor

@stephanvierkant stephanvierkant commented May 24, 2024

Q A
Branch? 3.3
Tickets n/a
License MIT
Doc PR n/a

I've added some more phpdoc annotations. Please let me know if I should add more. I've added just some annotations that are relevant for me own use cases.

@soyuka
Copy link
Member

soyuka commented May 24, 2024

sure but don't forget to run cs-fixer, phpstan and to respect the commit lint rules, thanks.

@stephanvierkant
Copy link
Contributor Author

I'll do! That's why I made it a draft PR, because I didn't run it yet.

@stephanvierkant stephanvierkant changed the title Add phpdocs to orm fix(metadata) Added phpdoc to ORM May 24, 2024
@stephanvierkant
Copy link
Contributor Author

Because 'Foo' wasn't recognized as a class-string, I've used Dummy::class instead.

@soyuka soyuka merged commit e06c88b into api-platform:3.3 May 31, 2024
74 of 76 checks passed
@soyuka
Copy link
Member

soyuka commented May 31, 2024

nice thanks!

@stephanvierkant stephanvierkant deleted the phpdoc branch May 31, 2024 09:22
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.

2 participants