Closed motoras closed 6 years ago
Thank you. I have merged it.
As I was discussing with Tony this morning, my main concern is identifying the functionality we are missing, and plan who to assign it to via github issues. Once we have at least one implementation of everything we can look at how to merge them etc. In particular I will be looking at the YAML tests illustrating the business requirements at the protocol level. ᐧ
On 27 February 2018 at 08:18, motoras notifications@github.com wrote:
@motoras https://github.com/motoras requested your review on: OpenHFT/Chronicle-Accelerate#6 https://github.com/OpenHFT/Chronicle-Accelerate/pull/6 My latest work.mostley on exchange stuff.
— You are receiving this because your review was requested. Reply to this email directly, view it on GitHub https://github.com/OpenHFT/Chronicle-Accelerate/pull/6#event-1493784476, or mute the thread https://github.com/notifications/unsubscribe-auth/ABBU8dOu7UpQ_ZtMvQ2VnOoDUVA7iPakks5tY7pCgaJpZM4SUhTm .
--
Peter Lawrey CEO +44 7557017054 <07771%20662994> chronicle.software
Trading as Chronicle Software Ltd
This is a merge of my latest work together withthe last head from the master. I had to fix a lot of conflicts, but this version, runs properly my work, and is passing the tests for the api and server projects. Due to the conflicting changes the auction project will not compile. It is really improtant to get a feedback on this work, as I have no idea how I should proceed forward.