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

Backport NonProxyLoadingEntityManager::refresh() signature change #10197

Merged
merged 1 commit into from
Nov 2, 2022

Conversation

michnovka
Copy link
Contributor

BP as requested in #10196

@derrabus derrabus added this to the 2.14.0 milestone Nov 2, 2022
@derrabus derrabus changed the title 2.14.x backport PR 10196 Backport NonProxyLoadingEntityManager::refresh() signature change Nov 2, 2022
@derrabus
Copy link
Member

derrabus commented Nov 2, 2022

btw, the email address that you use when authoring your commits is not connected to your GitHub account. You should fix that if you want your contributions to be counted.

@michnovka
Copy link
Contributor Author

@derrabus what do you mean, what e-mail do u see when authoring the commits?

@michnovka
Copy link
Contributor Author

OK, I understand. Will fix for future commits

@derrabus
Copy link
Member

derrabus commented Nov 2, 2022

If you want to change the author of the commit in this PR, please do so. I can wait with the merge.

You can also register more email adresses with your account on https://github.com/settings/emails. That should give you credit for the commits I've already merged.

@michnovka
Copy link
Contributor Author

Ive added my email to the github, you can merge. Im doing it for the humanity, not the credits anyways ;]

@derrabus derrabus merged commit c97f0a1 into doctrine:2.14.x Nov 2, 2022
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.

None yet

2 participants