Skip to content

Minor docblock spelling correction in AbstractClassMetadataFactory #716

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

Merged
merged 1 commit into from
Feb 1, 2017

Conversation

gnat42
Copy link
Contributor

@gnat42 gnat42 commented May 5, 2016

Just a minor docblock typo

Just a minor docblock typo
@stof stof merged commit 3a11c5a into doctrine:master Feb 1, 2017
@Ocramius Ocramius self-assigned this Feb 1, 2017
@stof
Copy link
Member

stof commented Feb 1, 2017

Good catch. Thanks

@greg0ire
Copy link
Member

greg0ire commented Feb 1, 2017

Can you make a release? :trollface:

@Ocramius Ocramius assigned stof and unassigned Ocramius Feb 1, 2017
@stof
Copy link
Member

stof commented Feb 1, 2017

just for that ? not worth it IMO

@soullivaneuh
Copy link
Contributor

just for that ? not worth it IMO

@stof I think this is a very very very critical issue.

Really.

:trollface:

@greg0ire
Copy link
Member

greg0ire commented Feb 1, 2017

@stof I edited my post to make the trolling obvious :P

@Ocramius
Copy link
Member

Ocramius commented Feb 1, 2017

Send critical money to my address for a release :-P

@soullivaneuh
Copy link
Contributor

soullivaneuh commented Feb 1, 2017

@Ocramius I'm from Lille, we pay only with 🍻 here.

@Ocramius
Copy link
Member

Ocramius commented Feb 1, 2017

I could probably replace lunch with beer, but my current client wouldn't approve of my behavior in the office afterwards.

@gnat42 gnat42 deleted the patch-1 branch February 1, 2017 17:15
@greg0ire
Copy link
Member

greg0ire commented Feb 1, 2017

my current client

The one that forces you to work on Windows? They sound like fun!

@Ocramius Ocramius changed the title Minor docblock spelling correction Minor docblock spelling correction in AbstractClassMetadataFactory Jul 22, 2017
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.

5 participants