Support for a magic comment to temporarily disable a rule

[nogweii]

Given something like:

package 'openjdk-6-jre-lib' do
  action :install
end

package 'openjdk-6-jre' do
  action :install
end

include_recipe 'java'

This triggers FC005. Rightly so, but in this one particular case, I actually like the additional verbosity. I don't want to disable FC005 entirely in this cookbook, as I do want to be notified about violations in other recipes (or even elsewhere in the file). So I'm proposing this syntax (inspired by rubocop's):

# foodcritic:disable FC005
package 1 do
  # ...
end
package 2 do
  # ...
end
# foodcritic:enable FC005

[guillaume-brossard]

+1

[corniste01]

This would also help those of us who break the build on a foodcritic limit to mark issues that have an open foodcritic issue. The addition of an optional reason field would also allow us to explain why we're disabling a rule at that point. That would let us track back to the foodcritic issue and remove the disabling of the rule when it was fixed.

[lamont-granquist]

We should have this with # ~FC005 comments