These are chat archives for freya-fs/arachne

17th
Aug 2015
Tony Williams
@TWith2Sugars
Aug 17 2015 12:53
Right - I've managed to fix all the edge cases I stumbled across.
First time playing with fparsec as well
Ryan Riley
@panesofglass
Aug 17 2015 14:02
Thanks!
Tony Williams
@TWith2Sugars
Aug 17 2015 14:04
No problem, if this pr is decent enough for you guys how long do you think it will take to be pushed to nuget?
Ryan Riley
@panesofglass
Aug 17 2015 14:15
Getting the fix to NuGet is pretty quick. We have to wait for AppVeyor to finish building the package, then we can push to NuGet.
@TWith2Sugars I left a few comments of feedback. What do you think?
Tony Williams
@TWith2Sugars
Aug 17 2015 14:40
Nice, will look at them now
Tony Williams
@TWith2Sugars
Aug 17 2015 14:55
replied to some, currently trying an empty query string
Tony Williams
@TWith2Sugars
Aug 17 2015 15:06
Written a new test and changed to the code for it to pass
Tony Williams
@TWith2Sugars
Aug 17 2015 15:57
@panesofglass got your feed back, added some of my own
Tony Williams
@TWith2Sugars
Aug 17 2015 16:55
@panesofglass - Check out the latest changes.
Ryan Riley
@panesofglass
Aug 17 2015 17:52
@kolektiv, changes look good to me. Thoughts?