00:03 <FromGitter> <watzon> @jgillich if it's a reserved keyword I'd recommend either prefixing it with something, like `redis_return` or shortening it/spelling it a different way like `ret`. Is return the only command like that?

00:04 <FromGitter> <tenebrousedge> they mentioned `return`, `with`, and `union`

00:08 <FromGitter> <jgillich> Yep those are the ones. The idea is to have a nice interface similar to the html_builder shard, so I'm not a fan of prefixing all methods. I'll look into macros, but I agree with you guys, probably a bad idea. Although they'd only be overwritten in the query builder block and there's no reason to return from that anyway...

00:09 <FromGitter> <Blacksmoke16> got an example of it not working? are ways around certain cases

00:09 <FromGitter> <watzon> Yeah Lucky's HTML builder ended up having to make some compromises too, like `para` instead of `p`

00:09 <FromGitter> <watzon> Just make sure you document differences and you should be ok

00:20 <FromGitter> <tenebrousedge> wow

00:22 <FromGitter> <tenebrousedge> okay, nvm, just weird `icr` behavior

00:24 <FromGitter> <tenebrousedge> https://carc.in/#/r/851k

00:24 <FromGitter> <tenebrousedge> in `play` you get an error, in `icr` all methods subsequently return 3

00:25 <FromGitter> <Blacksmoke16> prob makes sense knowing how it works

00:27 <FromGitter> <tenebrousedge> I suspect that it would be best to report it as a bug, but...I ain't gonna

01:45 <FromGitter> <tenebrousedge> lovely. So I based my AoC2019 day 3 solution off a Python one. I noted a comment by it: ⏎ ⏎ > How long did it take your program to run? My program took 10 minutes. ⏎ Crystal: 418.7 milliseconds (not with `--release`) [https://gitter.im/crystal-lang/crystal?at=5de8613b1659720ca8fe0c5d]

01:48 <FromGitter> <tenebrousedge> nothing very interesting about it (https://github.com/tenebrousedge/aoc2019-cr/blob/master/src/3/first.cr)

02:00 <Stephie> trying to avoid entering a rant about macos every 10 seconds on the github issue tracker

02:00 <Stephie> it's trult fustrating how many bugs are just purely apple's fault

02:03 <FromGitter> <tenebrousedge> example?

02:40 <FromGitter> <watzon> I wish that the converter for JSON::Field could just be a proc

02:40 <FromGitter> <watzon> Maybe have a to_converter and a from_converter

02:40 <FromGitter> <watzon> Because I rarely want to go define a whole module and look up how to do it all over again

02:41 <FromGitter> <tenebrousedge> make it so, number one!

02:41 <FromGitter> <tenebrousedge> is there an `Integer#digits` equivalent, and it not, why?

02:46 <FromGitter> <Blacksmoke16> https://github.com/crystal-lang/crystal/issues/8554

02:48 <FromGitter> <tenebrousedge> 😅 what an odd coincidence that many people have discovered the need for this at the same time

04:12 <FromGitter> <watzon> Is there a good built in way to indent a whole IO?

04:13 <FromGitter> <Daniel-Worrall> Lol Stephie, just wait till Windows support comes out. It'll dominate the macos issues

04:21 <FromGitter> <watzon> Oh it definitely wi;;

04:28 <FromGitter> <watzon> Answered my own question

04:29 <FromGitter> <watzon> ```code paste, see link``` [https://gitter.im/crystal-lang/crystal?at=5de8879105eec2433dd720b9]

04:29 <FromGitter> <watzon> This should be added to the standard library imo

04:29 <FromGitter> <watzon> I find myself needing it alot

04:31 <FromGitter> <tenebrousedge> and this is a case where the distinction between `^` and `\A` is important

04:34 <FromGitter> <tenebrousedge> @watzon why `[ \t]` and not `\s` ?

04:34 <FromGitter> <watzon> Actually that would be better

04:34 <FromGitter> <watzon> I copied this from activesupport

04:35 <FromGitter> <tenebrousedge> ah

04:35 <FromGitter> <tenebrousedge> well otherwise I think it's fine

04:36 <FromGitter> <watzon> I'm writing a class generator right now and a method like that really helps for making sure everything is properly indented

09:38 <FromGitter> <j8r> @watzon I would avoid monkey patching the stdlib as much as possible

09:38 <FromGitter> <watzon> In this case it's just a single script, not meant to be imported or used in anything else

09:38 <FromGitter> <j8r> because it's harder to know where the method comes from

09:38 <FromGitter> <watzon> Normally I wouldn't

09:39 <FromGitter> <j8r> ok

09:39 <FromGitter> <watzon> But I think it would be nice if the stdlib had a method like that

09:39 <FromGitter> <watzon> Don't know if anyone would be for it though

09:40 <FromGitter> <j8r> there is `ljust`

09:40 <FromGitter> <j8r> and `rjust`

09:41 <FromGitter> <watzon> I don't think those would work on something that's multiline though, would they?

09:41 <FromGitter> <watzon> The idea of the `indent` method is to have it work with a whole body of text, inserting space before each line

09:42 <FromGitter> <j8r> It's pretty simple to do, no sure it's worth it or common enough to put it in stdlib

09:43 <FromGitter> <watzon> *shrug*

09:43 <FromGitter> <j8r> each_line + String.build + ljust, done

09:43 <FromGitter> <watzon> I have found plenty of instances where I could use it, but you're right it's easy enough to do