Run rake tasks with interpreter warnings enabled

[ashmaroli]

Reaction to #1174 (comment)

We really should aim to get to 0 warnings because Rouge is extremely noisy.

/cc @mojavelinux

[pyrmont]

He wasn't kidding about it being noisy, huh?

[ashmaroli]

He wasn't kidding about it being noisy, huh?

Hehe.. yeah. 😃
I think one easy way towards resolving this is by configuring RuboCop to convert regex to use the %r() syntax with varying delimiters..

[pyrmont]

I thought it simpler to convert /../ to %r/.../. Have submitted #1180 with Rubocop's ambiguity warnings enabled.

[pyrmont]

I've also played around with turning on Ruby's warnings as this PR does. #1180 helps reduce the number somewhat but there's still a lot. I'm having a look at how feasible it is to fix them by hand (maybe too time consuming and not worth the effort while so many substantive PRs are still outstanding).

[mojavelinux]

I'd be happy to take a crack at fixing some of the warnings once you get to a stopping point.

[pyrmont]

@mojavelinux Thanks :) I'll let you know how I go.

[ashmaroli]

I volunteer to help with this if @mojavelinux needs a hand..

[ashmaroli]

I thought it simpler to convert /../ to %r/.../.

I'm okay with whatever it takes to keep the boat afloat.. 😛

[mojavelinux]

to keep the boat afloat...

...and be a good Ruby citizen.

[ashmaroli]

Opened #1184 to remove all those warning: instance variable @all not initialized lines

[ashmaroli]

@pyrmont I'm sticking to the disable warnings by default stance for now. Once we've reduced the warnings to a manageable level, I'll submit a PR to make it enabled by default..

[mojavelinux]

👍 The important point is that they're simple to enable ;)

[ashmaroli]

Thanks Dan 😃