Просмотр исходного кода

fix lint settings and lint errors

Yuki Takei 2 лет назад
Родитель
Сommit
22a604db0e

+ 0 - 13
packages/presentation/.eslintrc.js

@@ -1,18 +1,5 @@
 module.exports = {
   extends: [
     'weseek/react',
-    'weseek/typescript',
   ],
-  env: {
-  },
-  globals: {
-  },
-  settings: {
-    // resolve path aliases by eslint-import-resolver-typescript
-    'import/resolver': {
-      typescript: {},
-    },
-  },
-  rules: {
-  },
 };

+ 5 - 0
packages/preset-templates/.eslintrc.js

@@ -0,0 +1,5 @@
+module.exports = {
+  extends: [
+    'plugin:vitest/recommended',
+  ],
+};

+ 0 - 15
packages/preset-themes/.eslintrc.js

@@ -1,17 +1,2 @@
 module.exports = {
-  extends: [
-    'weseek/typescript',
-  ],
-  env: {
-  },
-  globals: {
-  },
-  settings: {
-    // resolve path aliases by eslint-import-resolver-typescript
-    'import/resolver': {
-      typescript: {},
-    },
-  },
-  rules: {
-  },
 };

+ 5 - 0
packages/remark-attachment-refs/.eslintrc.cjs

@@ -0,0 +1,5 @@
+module.exports = {
+  extends: [
+    'weseek/react',
+  ],
+};

+ 1 - 1
packages/remark-attachment-refs/src/client/components/AttachmentList.tsx

@@ -67,7 +67,7 @@ export const AttachmentList = ({
       : attachments.map((attachment) => {
         return <AttachmentLink key={attachment._id} attachment={attachment} inUse={false} />;
       });
-  }, [refsContext, isLoading, attachments]);
+  }, [isLoading, error, attachments, refsContext, renderNoAttachmentsMessage]);
 
   return <div className={styles['attachment-refs']}>{renderContents()}</div>;
 

+ 6 - 5
packages/remark-attachment-refs/src/client/components/ExtractedAttachments.tsx

@@ -32,7 +32,7 @@ export const ExtractedAttachments = React.memo(({
   const getAttachmentsFilteredByFormat = useCallback(() => {
     return attachments
       .filter(attachment => attachment.fileFormat.startsWith('image/'));
-  }, []);
+  }, [attachments]);
 
   const getClassesAndStylesForNonGrid = useCallback(() => {
     const { options } = refsContext;
@@ -95,7 +95,7 @@ export const ExtractedAttachments = React.memo(({
     return (options?.grid != null)
       ? getClassesAndStylesForGrid()
       : getClassesAndStylesForNonGrid();
-  }, []);
+  }, [getClassesAndStylesForGrid, getClassesAndStylesForNonGrid, refsContext]);
 
   // eslint-disable-next-line @typescript-eslint/no-unused-vars
   const renderExtractedImage = useCallback((attachment: IAttachmentHasId, index: number) => {
@@ -122,13 +122,14 @@ export const ExtractedAttachments = React.memo(({
     // }
 
     return (
-      <div key={attachment._id} style={containerStyles}
-      // onClick={onClick}
+      <div
+        key={attachment._id}
+        style={containerStyles}
       >
         <img src={attachment.filePathProxied} alt={alt} className={imageClasses.join(' ')} style={imageStyles} />
       </div>
     );
-  }, [refsContext]);
+  }, [getClassesAndStyles, refsContext]);
 
   // const renderCarousel = useCallback(() => {
   //   const { options } = refsContext;

+ 8 - 6
packages/remark-attachment-refs/src/client/components/Ref.tsx

@@ -24,12 +24,14 @@ const RefSubstance = React.memo(({
   const { data, error, isLoading } = useSWRxRef(pagePath, fileNameOrId, isImmutable);
   const attachments = data != null ? [data] : [];
 
-  return <AttachmentList
-    refsContext={refsContext}
-    isLoading={isLoading}
-    error={error}
-    attachments={attachments}
-  />;
+  return (
+    <AttachmentList
+      refsContext={refsContext}
+      isLoading={isLoading}
+      error={error}
+      attachments={attachments}
+    />
+  );
 });
 
 export const Ref = React.memo((props: Props): JSX.Element => {

+ 8 - 6
packages/remark-attachment-refs/src/client/components/RefImg.tsx

@@ -38,12 +38,14 @@ const RefImgSubstance = React.memo(({
   const { data, error, isLoading } = useSWRxRef(pagePath, fileNameOrId, isImmutable);
   const attachments = data != null ? [data] : [];
 
-  return <AttachmentList
-    refsContext={refsContext}
-    isLoading={isLoading}
-    error={error}
-    attachments={attachments}
-  />;
+  return (
+    <AttachmentList
+      refsContext={refsContext}
+      isLoading={isLoading}
+      error={error}
+      attachments={attachments}
+    />
+  );
 });
 
 export const RefImg = React.memo((props: Props): JSX.Element => {

+ 8 - 6
packages/remark-attachment-refs/src/client/components/Refs.tsx

@@ -33,12 +33,14 @@ const RefsSubstance = React.memo(({
   const { data, error, isLoading } = useSWRxRefs(pagePath, prefix, { depth, regexp }, isImmutable);
   const attachments = data != null ? data : [];
 
-  return <AttachmentList
-    refsContext={refsContext}
-    isLoading={isLoading}
-    error={error}
-    attachments={attachments}
-  />;
+  return (
+    <AttachmentList
+      refsContext={refsContext}
+      isLoading={isLoading}
+      error={error}
+      attachments={attachments}
+    />
+  );
 });
 
 export const Refs = React.memo((props: Props): JSX.Element => {

+ 8 - 6
packages/remark-attachment-refs/src/client/components/RefsImg.tsx

@@ -64,12 +64,14 @@ export const RefsImgSubstance = React.memo(({
   }, isImmutable);
   const attachments = data != null ? data : [];
 
-  return <AttachmentList
-    refsContext={refsContext}
-    isLoading={isLoading}
-    error={error}
-    attachments={attachments}
-  />;
+  return (
+    <AttachmentList
+      refsContext={refsContext}
+      isLoading={isLoading}
+      error={error}
+      attachments={attachments}
+    />
+  );
 });
 
 export const RefsImg = React.memo((props: Props): JSX.Element => {

+ 0 - 13
packages/remark-drawio/.eslintrc.cjs

@@ -1,18 +1,5 @@
 module.exports = {
   extends: [
     'weseek/react',
-    'weseek/typescript',
   ],
-  env: {
-  },
-  globals: {
-  },
-  settings: {
-    // resolve path aliases by eslint-import-resolver-typescript
-    'import/resolver': {
-      typescript: {},
-    },
-  },
-  rules: {
-  },
 };

+ 1 - 5
packages/remark-drawio/tsconfig.json

@@ -2,11 +2,7 @@
   "$schema": "http://json.schemastore.org/tsconfig",
   "extends": "../../tsconfig.base.json",
   "compilerOptions": {
-    "jsx": "react-jsx",
-
-    "baseUrl": ".",
-    "paths": {
-    }
+    "jsx": "react-jsx"
   },
   "include": [
     "src"

+ 1 - 1
packages/remark-lsx/.eslintrc.cjs

@@ -1,7 +1,7 @@
 module.exports = {
   extends: [
     'weseek/react',
-    'weseek/typescript',
+    'plugin:vitest/recommended',
   ],
   env: {
   },

+ 1 - 1
packages/remark-lsx/src/server/routes/list-pages/add-num-condition.spec.ts

@@ -4,7 +4,7 @@ import { mock } from 'vitest-mock-extended';
 import { addNumCondition } from './add-num-condition';
 import type { PageQuery } from './generate-base-query';
 
-describe('addNumCondition() throws 400 http-errors instance ', () => {
+describe('addNumCondition() throws 400 http-errors instance', () => {
 
   it("when the param 'offset' is a negative value", () => {
 

+ 0 - 13
packages/ui/.eslintrc.cjs

@@ -1,18 +1,5 @@
 module.exports = {
   extends: [
     'weseek/react',
-    'weseek/typescript',
   ],
-  env: {
-  },
-  globals: {
-  },
-  settings: {
-    // resolve path aliases by eslint-import-resolver-typescript
-    'import/resolver': {
-      typescript: {},
-    },
-  },
-  rules: {
-  },
 };