refactor if conditions in prefer-default-export to improve readibility

Co-authored-by: Jordan Harband <ljharb@gmail.com>
import-js · Nov 26, 2022 · 8dfa229 · 8dfa229
1 parent c204a03
commit 8dfa229
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/src/rules/prefer-default-export.js b/src/rules/prefer-default-export.js
@@ -105,12 +105,9 @@ module.exports = {
         if (hasDefaultExport || hasStarExport || hasTypeExport) {
           return;
         }
-        if (target === 'single') {
-          if (specifierExportCount === 1) {
-            context.report(namedExportNode, SINGLE_EXPORT_ERROR_MESSAGE);
-          }
-        } else if (target === 'any') {
-          if (specifierExportCount > 0) {
+        if (target === 'single' && specifierExportCount === 1) {
+           context.report(namedExportNode, SINGLE_EXPORT_ERROR_MESSAGE);
+        } else if (target === 'any' && specifierExportCount > 0) {
             context.report(namedExportNode, ANY_EXPORT_ERROR_MESSAGE);
           }
         }