You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
What we want to do here is inject a custom runner, so it's nonsense to redefine checking.
Define a NewRunner function in RuleSet so that built-in rulesets always use that function. This allows custom rulesets to inject a custom runner by simply redefining the NewRunner(). No need to redefine Check().
The text was updated successfully, but these errors were encountered:
Some rulesets redefine
Check()
to support custom runners in rulesets. e.g.https://github.com/terraform-linters/tflint-ruleset-aws/blob/v0.16.1/aws/ruleset.go#L46-L59
https://github.com/terraform-linters/tflint-ruleset-google/blob/v0.19.0/google/ruleset.go#L46-L59
What we want to do here is inject a custom runner, so it's nonsense to redefine checking.
Define a
NewRunner
function in RuleSet so that built-in rulesets always use that function. This allows custom rulesets to inject a custom runner by simply redefining theNewRunner()
. No need to redefineCheck()
.The text was updated successfully, but these errors were encountered: