Allow specifying the default enable status of a check #484
Labels
Comments
This was referenced Jun 24, 2016
binford2k
added a commit
to binford2k/puppet-lint
that referenced
this issue
Jun 28, 2016
This commit adds back an 80chars check, but disables it by default. To do this, I also added the ability to re-enable checks that have been disabled. The spec is failing and I don't have time to fix it just now, but it works when called from the command line. I'll come back to it if nobody else picks this up. A better solution than calling `Puppet.configuration` to disable a check like I did might be to extend the `new_check` method. That's not orthagonal to this PR though. Supercedes rodjek#488 Fixes rodjek#484
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There are a number of new check requests being developed, some of which are not preferable to be enabled by default. Rather than forcing them out of puppet-lint and into an external gem, which forces someone to maintain the plugin and gem, improve the
new_checkfunction to include aenabledboolean.Currently, there appears to be no way to configure this without editing configuration.rb and adding a line like
self.settings['autoloader_layout_disabled'] = true. Direct editing of the defaults is not the preferred method to manage this.The text was updated successfully, but these errors were encountered: