Tidy-up tidy.rb

[ashmaroli]

This is mostly a matter of style.

• you can avoid multiple self. in a module by extending to its self
• explicit return is not required if that is the last expression in a block
• Jekyll::Hooks.register can be referred to as simply Hooks.register if they're within the Jekyll module.

Its all fine if you choose not to merge this in..

[ashmaroli]

:white_flag: Down boy, down..!

:meat_on_bone:

[wkirby]

@ashmaroli Go ahead and open this up again and we'll review. Houndbot is a bit crazy sometimes. Everything here looks fine to me, with the exception of the removal of the explicit return statements.

[wkirby]

@ashmaroli Looks good. We'll deploy a minor version bump of the gem sometime in the next couple weeks.

[ashmaroli]

Thanks. Have a few questions to ask..

• Apart from being more robust, is there any other advantage for these double checks?

  jekyll_config["jekyll_tidy"] && jekyll_config["jekyll_tidy"]["ignore_env"]

• Also how about passing site.config directly to Tidy.init instead of site and then compute further..