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.

Sign up for GitHub

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

Jump to bottom

Add missing use statement #9699

Merged
merged 2 commits into from Apr 30, 2022
Merged

Add missing use statement #9699

merged 2 commits into from Apr 30, 2022

Conversation

ThomasLandauer
Copy link
Contributor

No description provided.

@ThomasLandauer
Copy link
Contributor Author

Follow-up of #9696

@greg0ire greg0ire enabled auto-merge April 29, 2022 14:34
@greg0ire
Copy link
Member

Thanks @ThomasLandauer !

@ThomasLandauer ThomasLandauer mentioned this pull request Apr 29, 2022
Closed
@SenseException
Copy link
Member

@ThomasLandauer Thank you. Could you please target the branch 2.12.x and add missing use statements to the other examples too?

@greg0ire greg0ire changed the base branch from 2.11.x to 2.12.x April 29, 2022 20:46
@ThomasLandauer
Adding use to all other code blocks
cbafd4f 
Also unifying code style: No blank line between `<?php` and `use`
auto-merge was automatically disabled April 29, 2022 22:44

Head branch was pushed to by a user without write access

@ThomasLandauer
Copy link
Contributor Author

Done :-)
@greg0ire thanks for the rebase

@derrabus derrabus added this to the 2.12.2 milestone Apr 30, 2022
@derrabus derrabus merged commit a9309d7 into doctrine:2.12.x Apr 30, 2022
@ThomasLandauer ThomasLandauer deleted the patch-9 branch April 30, 2022 13:45
@github-actions github-actions bot mentioned this pull request May 3, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@derrabus derrabus derrabus approved these changes

Assignees
No one assigned
Labels
Documentation
Projects
None yet
Milestone
2.12.2
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@ThomasLandauer @greg0ire @SenseException @derrabus