Add disableFix secondary option to configuration object

[kuangrs]

What is the problem you're trying to solve?

Sometimes we need find error which should not be automatically fixed, or it was automatically fixed by mistake, instead, we want to fix it manually. So if closing autofix by passing property like disableFix in secondaryOptions will be very helpful

What solution would you like to see?

Add support for closing autofix by passing property like disableFix in secondaryOptions

in file lib/lintPostcssResult.js (stylelint version 13.13.1), line 105

stylelint/lib/lintPostcssResult.js

Lines 104 to 108 in 403248c

	fix:
	stylelintOptions.fix &&
	// Next two conditionals are temporary measures until #2643 is resolved
	isFileFixCompatible &&
	!postcssResult.stylelint.disabledRanges[ruleName],

change like this

ruleFunction(primaryOption, secondaryOptions, {
  fix:
      // Add support for closing autofix by passing property like `disableFix` in secondaryOptions
      !secondaryOptions.disableFix && 
      stylelintOptions.fix &&
      // Next two conditionals are temporary measures until #2643 is resolved
      isFileFixCompatible &&
      !postcssResult.stylelint.disabledRanges[ruleName],
   newline,
})(postcssRoot, postcssResult),

[jeddy3]

@kuangrs Thanks for the request and for using the template.

Just to confirm, you'd like to disable autofix on a per rule basis, like the severity secondary option? If so, sounds good to me. We even suggest plugin authors do it.

[kuangrs]

@kuangrs Thanks for the request and for using the template.

Just to confirm, you'd like to disable autofix on a per rule basis, like the severity secondary option? If so, sounds good to me. We even suggest plugin authors do it.

yes, I'd like to use it like serverity.

[jeddy3]

Thanks for confirming.

A new disableFix secondary option SGTM.

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

Please branch off v14 rather than master.