Camelia, the Perl 6 bug

IRC log for #november-wiki, 2009-08-13

| Channels | #november-wiki index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
07:56 szabgab joined #november-wiki
08:16 szabgab joined #november-wiki
08:16 arthur-_ joined #november-wiki
08:16 mj41 joined #november-wiki
08:16 omega joined #november-wiki
08:45 masak joined #november-wiki
09:21 sri_kraih joined #november-wiki
10:54 ihrd joined #november-wiki
10:54 ihrd left #november-wiki
11:36 ihrd joined #november-wiki
11:37 ihrd left #november-wiki
12:02 ruoso joined #november-wiki
12:16 masak arthur-_: pulled.
12:16 arthur-_ masak: thanks
12:17 masak arthur-_: if you discover a small typo and haven't pushed yet, there's "git commit --amend" to save you.
12:17 arthur-_ masak: ok noted
12:18 * masak reviews the commits themselves
12:19 arthur-_ masak: I found get '/hello/:name' => -> $n { a bit ugly
12:19 arthur-_ changed it to get '/hello/:name' answers -> $n {
12:19 arthur-_ if you have any better idea ...
12:19 masak I'll think about it.
12:20 masak I agree that '=> ->' doesn't read extremely well.
12:21 masak arthur-_: um, if I read the code right, it currently calls all handlers that match. is that intentional?
12:22 arthur-_ yep, i'll correct that once i start writing tests
12:22 masak nice.
12:23 masak I already wrote one spectest, but it presupposes a Web::MockRequest, which I haven't written yet.
12:23 arthur-_ ( i don't do request.finish() either )
12:26 masak hm, no.
12:26 masak I can fix that if you want.
12:27 arthur-_ i'll do it a bit later, i finish work in a few hours
12:27 masak oki.
12:28 masak in the meantime, I'll move lib/Astaire/Astaire.pm to lib/Astaire.pm, so that any script that includes Astaire won't have to do 'use Astaire::Astaire' :)
12:29 arthur-_ ok
12:30 masak (so, make sure to pull that change later)
14:49 szabgab joined #november-wiki
15:33 arthur-_ joined #november-wiki
15:50 arthur-_ masak: Web::Response.write wants Str, so post '/test' answers { 2 + 2 }; does not work
15:51 arthur-_ cat we say that write accepts Any ?
15:51 arthur-_ s/cat/can/
15:52 masak yes.
15:52 masak maybe simply remove the typing from that one.
15:52 arthur-_ ok, I do it
15:53 masak arthur-_++
15:58 * arthur-_ also makes Web::Response a rw as in racks spec, so we can haz 404
16:00 masak sorry, making what in Web::Response rw?
16:01 arthur-_ status
16:01 arthur-_ sorry :)
16:01 arthur-_ ok 9 - inexistant page gives 404 error
16:01 masak :)
16:01 masak nice.
16:02 masak s/inexistant/nonexistent/, though.
16:02 arthur-_ thanks
16:02 arthur-_ inexistant == french ^^
16:04 masak oui, I suspected as much. :)
16:19 arthur-_ masak: pull request
16:21 masak arthur-_: thank you. will look at it.
16:22 arthur-_ k.
16:24 masak hm, github says my fork queue is empty. I'll have to do it manually.
16:27 masak arthur-_: hm, your commits seem to recreate the astaire.pod file with a small 'a'.
16:27 masak nowadays, it's called Astaire.pod, for spectest reasons.
16:27 arthur-_ can you remove it ?
16:27 masak also, I'd recommend putting tests in t/spec, and not directly in t/
16:28 arthur-_ i did not touch it intentionally, probably just an error
16:28 masak ok, then I'll remove it.
16:28 arthur-_ ok about t/spec, i move the file
16:28 masak great.
16:29 masak there, pulled, changed, and pushed.
16:30 masak arthur-_: I just want to say that it's really nice working with you. :)
16:32 arthur-_ masak: thx ^^
16:33 arthur-_ arthur@aquarelle ~/d/s/c/web> git rm t/astaire/
16:33 arthur-_ fatal: pathspec 't/astaire/' did not match any files
16:33 arthur-_ ( rm -r is not better )
16:34 arthur-_ ( nice working with you too )
16:35 masak I removed it for you, so you just need to pull.
16:35 masak gotta go now.
16:35 arthur-_ ok
16:35 masak we're going swimming.
16:35 arthur-_ have fun
16:35 masak thanks.
16:35 masak see you around!
16:35 arthur-_ yep
19:14 sri_kraih joined #november-wiki
20:19 lichtkind joined #november-wiki
22:15 p6eval joined #november-wiki
23:44 ihrd joined #november-wiki

| Channels | #november-wiki index | Today | | Search | Google Search | Plain-Text | summary