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
Provide container-aware factories #57
Provide container-aware factories #57
Conversation
], | ||
] | ||
] | ||
]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
grrr :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
arrrgh, I checked the template, but this f** space is added by PHPStrom. In my defense I have to say that the PR is not ready yet ;). I'll fix it with my next commit. I think I have to investigate some time to set up php-cs-fixer ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a present for you: prooph/event-store#62
4a5af7d
to
cd9e4cb
Compare
cd9e4cb
to
b56b328
Compare
], | ||
], //EO service_bus | ||
], //EO prooph | ||
]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Travis will bleat! ;) hehe
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope. Please add config dir to .php_cs file
Provide container-aware factories
This PR is not completed yet.
Missing parts: