Add support to use --fix with checkAgainstRule

[aaronccasanova]

What is the problem you're trying to solve?

The --fix option does not work with the checkAgainstRule utility:

stylelint.utils.checkAgainstRule(
  {
    ruleName: 'length-zero-no-unit'
    ruleSettings: true,
    root,
  },
  (warning) => { /* Don't report if the problem can be automatically fixed */ },
)

What solution would you like to see?

I would like to be able to pass through the context.fix option to checkAgainstRule:

module.exports = createPlugin(
  'custom/length-zero-no-unit',
  (primary, secondary, context) => (root) => {
    stylelint.utils.checkAgainstRule(
      {
        ruleName: 'length-zero-no-unit'
        ruleSettings: true,
        fix: context.fix,
        root,
      },
      (warning) => { /* Don't report if the problem can be automatically fixed */ },
    )
  },
})

My team currently has this behavior patched with the following updates to checkAgainstRule:

stylelint/lib/utils/checkAgainstRule.js

Line 47 in 64b022a

rule(settings[0], /** @type {O} */ (settings[1]), {})(options.root, tmpPostcssResult);

- rule(settings[0], /** @type {O} */ (settings[1]), {})(options.root, tmpPostcssResult);
+ rule(settings[0], /** @type {O} */ (settings[1]), {fix: options.fix})(options.root, tmpPostcssResult);

[ybiquitous]

@aaronccasanova Thanks for the proposal. I agree with passing the fix option to checkAgainstRule(). Furthermore, giving a context object seems to make sense instead of fix. What do you think?

[aaronccasanova]

Great idea, passing through the entire PluginContext is definitely more future facing 👍

[ybiquitous]

@aaronccasanova Thanks for the feedback. Let's add the context parameter to checkAgainstRule.

I've labeled the issue as ready to implement. Please consider contributing if you have time.