These are chat archives for akkadotnet/AkkaStreams

14th
Jun 2016
Chris Constantin
@cconstantin
Jun 14 2016 02:54
@Silv3rcircl3 I’ve added the missing tests, I’ll move the classes from util next and push
will let you know when done
Chris Constantin
@cconstantin
Jun 14 2016 04:29
Marc Piechura
@marcpiechura
Jun 14 2016 04:46
Great, thx
Chris Constantin
@cconstantin
Jun 14 2016 04:48
I’ll work on updating the build with packaging the TCK tomorrow. Also the readme for the TCK
btw, based on https://msdn.microsoft.com/en-us/library/ms229043(v=vs.110).aspx, shouldn’t we name the project/package “Reactive.Streams.Tck”?
Marc Piechura
@marcpiechura
Jun 14 2016 05:11
Yeah, I don't know, it's like with System.IO, you use everywhere TCK
So it's fine for me to also use it that way in the namespace
Chris Constantin
@cconstantin
Jun 14 2016 05:16
two letter acronyms are special :) as per the document
anyway, we can leave as is
Marc Piechura
@marcpiechura
Jun 14 2016 05:18
Hehe yeah it's not a strong opinion based on rules, just looks better to me ;)
Chris Constantin
@cconstantin
Jun 14 2016 05:20
k
Marc Piechura
@marcpiechura
Jun 14 2016 14:58
@cconstantin have addressed some remarks, method names, folder structure and nuget package
readme.md for the tck is still missing
Chris Constantin
@cconstantin
Jun 14 2016 16:02
Alright, will try add the readme later today
Marc Piechura
@marcpiechura
Jun 14 2016 16:09
Cool, is there anything else ?
Chris Constantin
@cconstantin
Jun 14 2016 17:51
don’t think so, should finally be able to merge and release
Marc Piechura
@marcpiechura
Jun 14 2016 18:20
nice, then we only need a build setup in teamcity + dev branch if we want to follow the same procedure as akka