search

VirtualLiveLab / js-config

A config library for js projects in Virtual Live Lab
5 stars 0 forks source link

eslint: 条件分岐をより安全に書かせる #97

Closed sushi-chaaaan closed 4 months ago

sushi-chaaaan commented 4 months ago

https://zenn.dev/manalink_dev/articles/eslint-react-expression

を参考に、条件式に意図せぬfalsyな値が入りにくいルールに変更する

sushi-chaaaan commented 4 months ago

https://typescript-eslint.io/rules/strict-boolean-expressions/ を用いる

sushi-chaaaan commented 4 months ago

デフォルトのOptionは

const defaultOptions: Options = [
  {
    allowString: true,
    allowNumber: true,
    allowNullableObject: true,
    allowNullableBoolean: false,
    allowNullableString: false,
    allowNullableNumber: false,
    allowNullableEnum: false,
    allowAny: false,
    allowRuleToRunWithoutStrictNullChecksIKnowWhatIAmDoing: false,
  },
];

だが、0とNaNを条件式に入れるのを防ぎたいのでallowNumber: falseで上書きする