Misc updates and "bugfixes"

[brewcore]

These are some quick changes we found useful. Not really bugs, but some of these do break the build unnecessarily.

• Updated Phing to 2.6.
• Stopped failing on debug code detection because many popular modules (views specifically) include debugging options.
• Excluded coder module from php file set for analysis because it contains bad code, but is useful.

[xendk]

The reason the rule fails instead of only warning, is that we work with different assumptions. We don't check core and contrib modules, as it's not code we wrote. We put all contrib code in contrib/ or libraries/, which is two patterns that's explicitly ignored in phing-drupal-template.

The successor to phing-drupal-template, https://github.com/reload/drake_ci , works on the same assumption.

Any particular reason for including contrib in the checking?

[brewcore]

That makes sense. We like to check pretty much every piece of code on a site so that we're aware of any potential issues a contributed module may introduce. I get that not everyone would want that workflow.

In the future we're hoping to build a system that we can use to test community code and updates before deploying them to sites.