feat: warn if the primary option is false

lib/utils/__tests__/validateOptions.test.js

@@ -26,17 +26,17 @@ describe('validateOptions for primary options', () => {
 		expect(result.warn.mock.calls[0][0]).toBe('Invalid option value "d" for rule "foo"');
 	});
 
-	it('passing boolean equivalence', () => {
+	it('passing null equivalence', () => {
 		validateOptions(result, 'foo', {
-			possible: [true, false],
-			actual: false,
+			possible: [true, null],
+			actual: null,
 		});
 		expect(result.warn).toHaveBeenCalledTimes(0);
 	});
 
-	it('failing boolean equivalence', () => {
+	it('failing null equivalence', () => {
 		validateOptions(result, 'foo', {
-			possible: [true, false],
+			possible: [true, null],
 			actual: 'a',
 		});
 		expect(result.warn).toHaveBeenCalledTimes(1);
@@ -204,7 +204,7 @@ describe('validateOptions for `*-no-*` rule with no valid options', () => {
 		});
 		expect(result.warn.mock.calls[0]).toHaveLength(2);
 		expect(result.warn.mock.calls[0][0]).toBe(
-			'Unexpected option value "false" for rule "no-dancing"',
+			'Invalid option value "false" for rule "no-dancing". Are you trying to disable this rule? If so use "null" instead',
 		);
 	});
 

lib/utils/validateOptions.js

@@ -65,6 +65,17 @@ function validate(opts, ruleName, complain) {
 	const possible = opts.possible;
 	const actual = opts.actual;
 	const optional = opts.optional;
+	const whitelist = [
+		'reportDescriptionlessDisables',
+		'reportInvalidScopeDisables',
+		'reportNeedlessDisables',
+	];
+
+	if (actual === false && !whitelist.includes(ruleName)) {
+		return complain(
+			`Invalid option value "false" for rule "${ruleName}". Are you trying to disable this rule? If so use "null" instead`,
+		);
+	}
 
 	if (actual === null || arrayEqual(actual, [null])) {
 		return;