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 async message producer interface #67

Merged
merged 1 commit into from
Aug 30, 2015

Conversation

codeliner
Copy link
Member

@codeliner
Copy link
Member Author

ping @prolic

@sandrokeil
Copy link
Member

What naming conventions are you using? ZF2 uses the suffix Interface for Interfaces. So you know that it is an interface and you don't have to use a new name for a concrete implementation. What do you think about MessageProducerInterface?

@codeliner
Copy link
Member Author

hehe, prolic had the same question a few days/weeks ago. Please see: prooph/event-store#59 (comment)

So you know that it is an interface and you don't have to use a new name for a concrete implementation.

IMO it is an advantage that I have to think about a good name for a concrete implementation.
See: http://verraes.net/2013/09/sensible-interfaces/ .
I absolutely agree with Mathias and it is a great article about the topic.

@codeliner codeliner added this to the 4.2 Release milestone Aug 30, 2015
prolic added a commit that referenced this pull request Aug 30, 2015
Add async message producer interface
@prolic prolic merged commit 7eb8576 into prooph:master Aug 30, 2015
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.

3 participants