Hi Fujita-San, Thank you for reviewing my patch.
On 2016年10月29日 07:03, FUJITA Tomonori wrote: > On Thu, 27 Oct 2016 10:39:24 +0900 > Iwase Yusuke <[email protected]> wrote: > >>> Great! But running this unconditionally with Docker installed sounds >>> too big burden. How about having this as a script for CI instead of >>> UT? >> >> This UT will be skipped if Docker has not been installed, but still >> burden? >> Also, this UT is depending on only Docker (service and CLI), but not >> depending on other extra requirements. >> >> The reason I put this into UT is the ease of visualizing the coverages >> including other test cases at https:/coveralls.io. >> Having scripts is also available, but can we combine the coverages of >> UTs and these scripts? > > Ok, let's merge this and see if others would complain. Can you update > this for the current master? Sure, I will post it soon. Regards, Iwase ------------------------------------------------------------------------------ The Command Line: Reinvented for Modern Developers Did the resurgence of CLI tooling catch you by surprise? Reconnect with the command line and become more productive. Learn the new .NET and ASP.NET CLI. Get your free copy! http://sdm.link/telerik _______________________________________________ Ryu-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ryu-devel
