README.md suggests to use deprecated API

bennidi / mbassador

Powerful event-bus optimized for high throughput in multi-threaded applications. Features: Sync and Async event publication, weak/strong references, event filtering, annotation driven

MIT License

958 stars 146 forks source link

README.md suggests to use deprecated API #82

Closed holgerbrandl closed 10 years ago

holgerbrandl commented 10 years ago

The README.md suggests to use new MBassador(BusConfiguration.SyncAsync()), however SyncAsync seems to be deprecated in v1.2.

Most APIs provide a default constructor (or a factory that gives an instance without providing complex arguments), so maybe MBassador could do as well? :-)

aymeric-soubrouillard commented 10 years ago

"1.2.1

Not yet released! API-Changes: Removed deprecated method BusConfiguration.SyncAsync() -> use MBassador default constructor instead"

Is that what you was looking for?

holgerbrandl commented 10 years ago

@aymeric-soubrouillard yes indeed, it's exactly what I meant. Thank you for fixing it and for providing this great library.

aymeric-soubrouillard commented 10 years ago

I am not the developper, i am just a user like you. But yes, thanks to Benjamin Diedrichsen for Mbassador.

holgerbrandl commented 10 years ago

Sure, sorry for my confusion. Thanks Benjamin Diedrichsen for providing this great library!

bennidi commented 10 years ago

Yes, hail to Benjamin Diedrichsen :) @aymeric-soubrouillard thanks for pointing out the changes I had made over the weekend. A new release will come in December. I think it will mainly include updated documentation and validation of handler and bus configuration.