enable unicorn/prefer-logical-operator-over-ternary rule

graphql · Dec 2, 2022 · 04742d3 · 04742d3
1 parent f7addb2
commit 04742d3
Show file tree

Hide file tree

Showing 7 changed files with 13 additions and 12 deletions.
diff --git a/.changeset/dirty-turkeys-study.md b/.changeset/dirty-turkeys-study.md
@@ -0,0 +1,7 @@
+---
+'codemirror-graphql': patch
+'graphiql': patch
+'graphql-language-service': patch
+---
+
+enable `unicorn/prefer-logical-operator-over-ternary` rule
diff --git a/packages/codemirror-graphql/src/__tests__/lint-test.ts b/packages/codemirror-graphql/src/__tests__/lint-test.ts
@@ -19,7 +19,7 @@ import { TestSchema } from './testSchema';
 function createEditorWithLint(lintConfig?: any) {
   return CodeMirror(document.createElement('div'), {
     mode: 'graphql',
-    lint: lintConfig ? lintConfig : true,
+    lint: lintConfig || true,
   });
 }
 

diff --git a/packages/codemirror-graphql/src/variables/__tests__/lint-test.ts b/packages/codemirror-graphql/src/variables/__tests__/lint-test.ts
@@ -19,7 +19,7 @@ import '../mode';
 function createEditorWithLint(lintConfig?: any) {
   return CodeMirror(document.createElement('div'), {
     mode: 'graphql-variables',
-    lint: lintConfig ? lintConfig : true,
+    lint: lintConfig || true,
   });
 }
 

diff --git a/packages/graphiql/postcss.config.js b/packages/graphiql/postcss.config.js
@@ -2,9 +2,7 @@ module.exports = ({ file, options }) => ({
   plugins: {
     'postcss-import': { root: file.dirname },
     // contains autoprefixer, etc
-    'postcss-preset-env': options['postcss-preset-env']
-      ? options['postcss-preset-env']
-      : false,
+    'postcss-preset-env': options['postcss-preset-env'] || false,
     cssnano: process.env.NODE_ENV === 'production' ? options.cssnano : false,
   },
 });
diff --git a/packages/graphiql/src/components/GraphiQL.tsx b/packages/graphiql/src/components/GraphiQL.tsx
@@ -303,9 +303,7 @@ export function GraphiQLInterface(props: GraphiQLInterfaceProps) {
       <ToolbarButton onClick={() => copy()} label="Copy query (Shift-Ctrl-C)">
         <CopyIcon className="graphiql-toolbar-icon" aria-hidden="true" />
       </ToolbarButton>
-      {props.toolbar?.additionalContent
-        ? props.toolbar.additionalContent
-        : null}
+      {props.toolbar?.additionalContent || null}
     </>
   );
 

diff --git a/packages/graphql-language-service/src/interface/getHoverInformation.ts b/packages/graphql-language-service/src/interface/getHoverInformation.ts
@@ -232,7 +232,7 @@ function renderDeprecation(
     return;
   }
 
-  const reason = def.deprecationReason ? def.deprecationReason : null;
+  const reason = def.deprecationReason || null;
   if (!reason) {
     return;
   }

diff --git a/packages/graphql-language-service/src/parser/CharacterStream.ts b/packages/graphql-language-service/src/parser/CharacterStream.ts
@@ -53,9 +53,7 @@ export default class CharacterStream implements CharacterStreamInterface {
   public sol = (): boolean => this._pos === 0;
 
   public peek = (): string | null => {
-    return this._sourceText.charAt(this._pos)
-      ? this._sourceText.charAt(this._pos)
-      : null;
+    return this._sourceText.charAt(this._pos) || null;
   };
 
   public next = (): string => {
-Original file line number
+Diff line change
@@ Expand Up / @@ -232,7 +232,7 @@ function renderDeprecation( @@
         return;
       }
-      const reason = def.deprecationReason ? def.deprecationReason : null;
+      const reason = def.deprecationReason || null;
       if (!reason) {
         return;
       }
@@ Expand Down @@