Skip to main content

no-unnecessary-boolean-literal-compare

Disallow unnecessary equality comparisons against boolean literals.

Comparing boolean values to boolean literals is unnecessary, those comparisons result in the same booleans. Using the boolean values directly, or via a unary negation (!value), is more concise and clearer.

Attributes

  • Included in configs
    • ✅ Recommended
    • 🔒 Strict
  • Fixable
    • 🔧 Automated Fixer
    • 🛠 Suggestion Fixer
  • 💭 Requires type information

Rule Details

This rule ensures that you do not include unnecessary comparisons with boolean literals. A comparison is considered unnecessary if it checks a boolean literal against any variable with just the boolean type. A comparison is not considered unnecessary if the type is a union of booleans (string | boolean, someObject | boolean).

Warning: Do not use this rule when strictNullChecks is disabled. ESLint is not able to distinguish between false and undefined or null values. This can cause unintended code changes when using autofix.

Note: Throughout this page, only strict equality (=== and !==) are used in the examples. However, the implementation of the rule does not distinguish between strict and loose equality. Any example below that uses === would be treated the same way if == was used, and any example below that uses !== would be treated the same way if != was used.

Examples of code for this rule:

declare const someCondition: boolean;
if (someCondition === true) {
}

Options

The rule accepts an options object with the following properties.

type Options = {
// if false, comparisons between a nullable boolean variable to `true` will be checked and fixed
allowComparingNullableBooleansToTrue?: boolean;
// if false, comparisons between a nullable boolean variable to `false` will be checked and fixed
allowComparingNullableBooleansToFalse?: boolean;
};

Defaults

This rule always checks comparisons between a boolean variable and a boolean literal. Comparisons between nullable boolean variables and boolean literals are not checked by default.

const defaults = {
allowComparingNullableBooleansToTrue: true,
allowComparingNullableBooleansToFalse: true,
};

allowComparingNullableBooleansToTrue

Examples of code for this rule with { allowComparingNullableBooleansToTrue: false }:

declare const someUndefinedCondition: boolean | undefined;
if (someUndefinedCondition === true) {
}

declare const someNullCondition: boolean | null;
if (someNullCondition !== true) {
}

allowComparingNullableBooleansToFalse

Examples of code for this rule with { allowComparingNullableBooleansToFalse: false }:

declare const someUndefinedCondition: boolean | undefined;
if (someUndefinedCondition === false) {
}

declare const someNullCondition: boolean | null;
if (someNullCondition !== false) {
}

Fixer

ComparisonFixer OutputNotes
booleanVar === truebooleanVar
booleanVar !== true!booleanVar
booleanVar === false!booleanVar
booleanVar !== falsebooleanVar
nullableBooleanVar === truenullableBooleanVarOnly checked/fixed if the allowComparingNullableBooleansToTrue option is false
nullableBooleanVar !== true!nullableBooleanVarOnly checked/fixed if the allowComparingNullableBooleansToTrue option is false
nullableBooleanVar === falsenullableBooleanVar ?? trueOnly checked/fixed if the allowComparingNullableBooleansToFalse option is false
nullableBooleanVar !== false!(nullableBooleanVar ?? true)Only checked/fixed if the allowComparingNullableBooleansToFalse option is false