Peder Stray

Results 16 comments of Peder Stray

"" is the glob operator or reading from a filehandle, don't think it needs spaces around it, as it usually is within a list context like () or part of...

Seems to be working better, but there still are some issues. - You don't want space around % if followed by { or $, as in `keys %$foo` and `keys...

I had a look at the code, and it seems `&=`, `&&=`, and `||=` are missing. `|.`, `&.`, and `^.` should have space around them. I also note that `$foox=`...

Well, the code that does the auto-detect only selects between default-markup (if the code starts with a

I think it would actually be better to just replace the whole highlight-engine with highlight.js, and and may bundle a few variants of the language-sets... it was pretty easy to...

Seems it does, but what it does not do is line-numbering... but that could probably be added by another js lib...

Ok, moved the message from Minion/Job.pm to Minion/Command/worker/minion.pm. Sounds like a much more appropriate place for it. Still no tests though, mostly because I have no idea how to test...

Removed one of the functions, _dequeue, and fixed the indentation and some spacing to match the other code. Still, a bit more than a one-liner, but that wouldn't be very...

This works just as expected in my config: `(:name "Important" :priority "A")`

I agree with @Cadair here. Would be nice to get rid of the "Other items" heading when it is the only one. Would be nice if any single header could...