Perl 6 - the future is here, just unevenly distributed

IRC log for #metacpan, 2014-07-11

| Channels | #metacpan index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
01:01 castaway joined #metacpan
01:38 FROGGS_ joined #metacpan
01:43 klapperl joined #metacpan
02:24 jibsheet if someone has a chance, can I get a reindex of RT::Extension::ReportSpam
02:30 ether_ joined #metacpan
03:27 talina_ joined #metacpan
04:31 trs jibsheet: done
05:01 oiami joined #metacpan
06:00 rashi joined #metacpan
07:15 neilb joined #metacpan
07:25 chankey joined #metacpan
07:57 neilb joined #metacpan
08:29 rashi joined #metacpan
08:31 neilb joined #metacpan
10:49 castaway joined #metacpan
11:14 ribasushi joined #metacpan
11:30 rashi joined #metacpan
11:31 ribasushi joined #metacpan
11:41 ribasushi joined #metacpan
12:36 rashi joined #metacpan
12:55 jibsheet trs: thanks
14:39 cooper joined #metacpan
15:27 alh joined #metacpan
16:22 oiami left #metacpan
16:25 FROGGS joined #metacpan
17:06 neilb joined #metacpan
17:21 metacpan joined #metacpan
17:21 metacpan [metacpan-web] tsibley created tsibley/module-name-search (+1 new commit): http://git.io/Be4PHA
17:21 metacpan metacpan-web/tsibley/module-name-search d385e52 Thomas Sibley: Search only by module name when a term is prefixed with "module:"...
17:21 metacpan left #metacpan
17:21 dipsy [ Search only by module name when a term is prefixed with "module:" · d385e52 · CPAN-API/metacpan-web · GitHub ]
17:28 neilb_ joined #metacpan
17:44 metacpan joined #metacpan
17:44 metacpan [metacpan-web] oalders pushed 1 new commit to master: http://git.io/Wwg_3Q
17:44 metacpan metacpan-web/master 304bff2 Olaf Alders: Merge pull request #1246 from CPAN-API/tsibley/module-name-search...
17:44 metacpan left #metacpan
17:44 metacpan joined #metacpan
17:44 metacpan [metacpan-web] oalders deleted tsibley/module-name-search at d385e52: http://git.io/Mig61g
17:44 metacpan left #metacpan
17:44 dipsy [ Merge pull request #1246 from CPAN-API/tsibley/module-name-search · 304bff2 · CPAN-API/metacpan-web · GitHub ]
17:44 dipsy [ Search only by module name when a term is prefixed with "module:" · d385e52 · CPAN-API/metacpan-web · GitHub ]
17:46 oalders trs++
17:46 oalders https://metacpan.org/search?q=module%3AMooseX%3A%3ATypes
17:46 dipsy [ Search for "module:MooseX::Types" - metacpan.org ]
18:26 rwstauner as a side effect, you can't search for "module::metadata" (though you can search for "Module::Metadata")
18:27 trs rwstauner: that can be fixed with a regex change. nice catch!
18:27 rwstauner :-)
18:51 metacpan joined #metacpan
18:51 metacpan [metacpan-web] tsibley created tsibley/stricter-special-query-terms (+2 new commits): http://git.io/4XjB5A
18:51 metacpan metacpan-web/tsibley/stricter-special-query-terms 22c0855 Thomas Sibley: Require a word character after the colon in module: searches...
18:51 metacpan metacpan-web/tsibley/stricter-special-query-terms 294ae97 Thomas Sibley: Only replace special search terms when they're a standalone word...
18:51 metacpan left #metacpan
18:51 dipsy [ Comparing 22c0855bf840^...294ae970a5c5 · CPAN-API/metacpan-web · GitHub ]
18:53 trs do make sure #1247 passes tests before merging.
19:01 oalders i waited on the build earlier as well :)
19:05 trs :)
19:05 trs wasn't sure
19:21 oalders trs: good thing, because it failed ;)
19:21 oalders do you have the pre-commit hook enabled?
19:36 DerAlex joined #metacpan
19:39 trs oalders: pushed a tidy fix.
19:39 metacpan joined #metacpan
19:39 metacpan [metacpan-web] tsibley force-pushed tsibley/stricter-special-query-terms from 294ae97 to 6a3a24c: http://git.io/bjqwSQ
19:39 metacpan metacpan-web/tsibley/stricter-special-query-terms 6a3a24c Thomas Sibley: Only replace special search terms when they're a standalone word...
19:39 metacpan left #metacpan
19:39 dipsy [ Commits · CPAN-API/metacpan-web · GitHub ]
19:41 trs I don't have the pre-commit hook on this repo
19:41 trs s/repo/clone/
19:50 punter joined #metacpan
19:57 trs man, I knew it was slower, but I didn't recognize just how much slower metacpan was until it was fixed yesterday.
19:57 trs and now it's zippy
20:00 trs oalders: so the thing with the hook (now installed in this clone) is that it prevents commit without tidy.
20:00 trs and it's more often the case that I want to tidy stuff in a separate commit.
20:00 trs so that the actual functional changes aren't lost in the noise.
20:00 oalders trs: git commit  --no-verify
20:18 trs oalders: nod.
20:18 trs there's also this problem: http://tsibley.net/paste/2014-07-118eJDFpx_-tidy-problems.html
20:18 trs which
20:19 trs is a non-starter for me because I very often selectively stage hunks.
20:19 oalders trs: you're referring to the weird stuff that goes on with the stash?
20:20 trs having to resolve conflicts after it automatically pops the stash it saved is annoying.
20:20 oalders yeah, i do hate that
20:20 oalders you should do what's best for your workflow
20:20 trs well, I do :)
20:20 oalders if you don't want the hook, turn it off :)
20:20 trs but I just wanted to explain why I don't usually have the hook enabled.
20:20 oalders if there's a problem, you'll find out about it eventually
20:20 trs and why you have to pester me every other commit about tidy failures :)
20:21 oalders :) it's valuable feedback
20:21 oalders you can also just run tidyall on the files after the commits if you want to check
20:21 trs well, s/valuable// at least ;)
20:21 oalders :)
20:21 oalders i've had that stash part bite me a few times and it's very confusing
20:21 trs yeah
20:22 trs took me a couple minutes the first time to figure out what it was doing and why I had a merge conflict.
20:22 trs and it still is confusing when it happens ("wtf, where'd my staged/unstaged changes go?!")
20:23 oalders i wonder if that's fixable
20:23 trs also, if you then use git mergetool to resolve the merge, everything is now staged, including the bit that was unstaged previously.
20:23 oalders i think i've done a hard reset at times just to get past it
20:23 trs rwstauner: bug report: https://metacpan.org/pod/Parse::ANSIColor::Tiny#remove_escape_sequences didn't get 'em all in this: http://tsibley.net/paste/2014-07-118eJDFpx_-tidy-problems.html
20:23 dipsy [ Parse::ANSIColor::Tiny - Determine attributes of ANSI-Colored string - metacpan.org ]
20:24 trs I can provide the source material if you'd like.
20:24 mst trs: I tend to prefer to perltidy before staging things
20:24 mst and to have a pre-commit hook that complains if anything changes
20:24 mst that seems to work out fine
20:24 trs mst: ... I think that's what the current metacpan workflow is for the hook.
20:25 trs unless I'm missing a detail?
20:25 oalders nope, it sounds like mst's workflow is similar
20:25 mst trs: so far as I can tell, your problem is that you're staging hunks *before* the perltidy hits
20:26 trs yes, because I don't want actual functional changes to be lost in tidy noise.
20:26 trs granted, it's less so when all your files are clean so the only bits that change are the ones you're touching.
20:26 oalders diff -w gets rid of a lot of noise
20:26 trs but still, comparison of the diff is impacted by line breaks.
20:26 trs or reindents
20:26 trs oalders: yep, and other ignore modes
20:27 rwstauner trs: i see some escapes in there that i know i'm not handling, so send me the source and i'll take a closer look
20:28 trs maybe I'm just too much of a pedant :)
20:28 mst trs: well, yes. you don't commit a diff that's tidying anything you didn't touch
20:28 trs rwstauner: OK.
20:28 mst trs: no, you're not enough of a pedant
20:28 mst trs: the trick is: tidy the file you're going to work on. commit the tidying. edit the file. tidy it again. commit tidied hunks against tidied file.
20:30 trs mst: yep, that workflow is obvious. I'll reiterate that my problem with it is that the "Tidy it again" step can still introduce noise to your hunks if your change pushes perltidy over the edge.
20:31 mst I thought normally 'over the edge' still only resulted in fairly localised changes
20:32 rwstauner i have all of the above issues as well, frequently, with the added frustration that i think "tidy" always does the opposite (makes the code harder to read)
20:32 trs they are fairly localized, but comparison of the diff is still impacted by added line breaks.
20:32 rwstauner but it might be better if the hook could simply abort or warn without staging and adding conflict markers
20:32 trs and I'm with rwstauner in that I think perltidy often makes it harder to read.
20:33 rwstauner b/c then i could catch it and choose what to do, rather than having to go in and undo the conflict markers
20:33 rwstauner so in the end it adds a lot more busywork
20:33 rwstauner i'd rather have it nag me than munge the source
20:33 rwstauner so that i could run the munger manually
20:34 rwstauner it would be faster/less work with the way i operate
20:34 oalders i'd like to see that change as well
20:35 mst trs: I don't always agree with perltidy's changes, but if you're using it at all, then consistency is useful
20:35 mst trs: also, I dislike that change-functionality-then-commit-tidying makes blame find the tidying, not the functionality change
20:35 mst trs: so now instead of having a slightly harder to read diff, you find completely the wrong diff. I don't consider that an improvement :)
20:41 trs mst: I use tig for blaming, which makes bouncing back another commit a single keypress.
20:41 trs mst: so less of a concern for me there, but understood.
20:43 trs mst: I agree that tidy before change is almost always preferable, but that is at odds with enforced tidying if you don't want to tidy your changed hunks in the same commit as them.
20:43 trs when tidying isn't enforced, but there's a policy for cleaning up particularly legacy code, tidy before change is great.
20:45 trs at the end of the day though, I'd rather be annoyed and following contributing guidelines for a project than throw up my hands and walk away :)
20:49 mst trs: basically, if metacpan has a tidying test, then it's effectively enforced tidying, and I think you should tidy them in the same commit to maximise history usefulness
20:49 mst trs: and that the process I gave pretty much does that, or as close to it as you can easily get
20:50 mst trs: I do, however, entirely support your right to grumble about the changes perltidy makes, because they annoy me sometimes too - but having weird history annoys me more :)
20:55 trs fair enough :)
21:41 neilb joined #metacpan
22:10 ether joined #metacpan
22:11 CletusSlackjawedYok joined #metacpan
22:17 ether joined #metacpan
22:34 punter joined #metacpan
23:01 neilb joined #metacpan
23:03 neilb joined #metacpan

| Channels | #metacpan index | Today | | Search | Google Search | Plain-Text | summary