I also think I've gone through most of them and back then all got applied or 
didn't cut it. 
I simply have no way to close them though...

LieGrue,
strub


> Am 05.06.2018 um 07:57 schrieb Romain Manni-Bucau <[email protected]>:
> 
> Hi guys,
> 
> spent some time reviewing our PRs on github (OWB+MW), think all have an up
> to date comment if needed (some are missing headers, one was introducing a
> bug, others were mainly good).
> 
> I'm not sure why i missed them on the list but some are old and should pby
> be better at merging them.
> 
> If anyone has some free time to do another pass and potentially merge a few
> (@reta one for instance is ready to merge) it can be very good too.
> I'll try to do another one if comments are fixed end of the week to try to
> merge a few.
> 
> Romain Manni-Bucau
> @rmannibucau <https://twitter.com/rmannibucau> |  Blog
> <https://rmannibucau.metawerx.net/> | Old Blog
> <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> |
> LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book
> <https://www.packtpub.com/application-development/java-ee-8-high-performance>

Reply via email to