From 31babb1a529c3769e8e4306ae3c7aa3b615c2785 Mon Sep 17 00:00:00 2001 From: Alexey Pyltsyn Date: Thu, 23 Jan 2020 20:36:21 +0300 Subject: [PATCH] chore(eslint): require curly brackets on all blocks --- .eslintrc.js | 1 + packages/docusaurus-theme-classic/src/theme/DocItem/index.js | 4 +++- packages/docusaurus/src/client/PendingNavigation.js | 4 +++- 3 files changed, 7 insertions(+), 2 deletions(-) diff --git a/.eslintrc.js b/.eslintrc.js index 0a6ec2c7be0e..aa09dca17d27 100644 --- a/.eslintrc.js +++ b/.eslintrc.js @@ -51,6 +51,7 @@ module.exports = { 'jsx-a11y/no-noninteractive-element-interactions': WARNING, 'no-console': OFF, 'no-underscore-dangle': OFF, + curly: [WARNING, 'all'], 'react/jsx-closing-bracket-location': OFF, // Conflicts with Prettier. 'react/jsx-filename-extension': OFF, 'react/jsx-one-expression-per-line': OFF, diff --git a/packages/docusaurus-theme-classic/src/theme/DocItem/index.js b/packages/docusaurus-theme-classic/src/theme/DocItem/index.js index 0cfc69964cc1..14577fb44cae 100644 --- a/packages/docusaurus-theme-classic/src/theme/DocItem/index.js +++ b/packages/docusaurus-theme-classic/src/theme/DocItem/index.js @@ -32,7 +32,9 @@ function DocTOC({headings}) { /* eslint-disable jsx-a11y/control-has-associated-label */ function Headings({headings, isChild}) { - if (!headings.length) return null; + if (!headings.length) { + return null; + } return (