style: fix all eslint errors (#491)

affects: @gio-design/components

package.json

@@ -55,7 +55,7 @@
     "build:website": "yarn workspace website build",
     "build-storybook": "yarn workspace @gio-design/components build-storybook",
     "g:icons": "yarn workspace @gio-design/icons generate",
-    "lint:components": "eslint packages/components/src --ext .js,.jsx,.ts,.tsx",
+    "lint:components": "eslint packages/components/src --ext .ts,.tsx",
     "lint:style": "stylelint 'packages/components/src/**/*.{css,less}' --syntax less",
     "test": "yarn workspace @gio-design/components test",
     "version": "npx lerna version --no-private --no-git-tag-version",

packages/components/src/components/banner/Banner.tsx

@@ -22,7 +22,7 @@ const Banner: React.FC<BannerProps> = (props: BannerProps) => {
         !visible && `${prefixCls}-close`
       )}
     >
-      <div className={className(`${prefixCls}-content`, button && `${prefixCls}-content-button`)}> {content}</div>
+      <div className={className(`${prefixCls}-content`, button && `${prefixCls}-content-button`)}>{content}</div>
       <div className={className(`${prefixCls}-button`)}>{button}</div>
       <div
         className={className(`${prefixCls}-closeIcon`)}

packages/components/src/components/banner/__tests__/Banner.spec.js

@@ -24,7 +24,7 @@ describe('Testing Banner', () => {
 
   it('props content', () => {
     const stringwrapper = mount(<Banner type="normal" content="content" />).children();
-    expect(stringwrapper.childAt(0).text()).toBe(' content');
+    expect(stringwrapper.childAt(0).text()).toBe('content');
     const nodeWrapper = mount(<Banner content={<div>content</div>} />).children();
     expect(nodeWrapper.childAt(0).type()).toBe('div');
   });
@@ -34,12 +34,10 @@ describe('Testing Banner', () => {
     expect(mount(<Banner type="alert" />).exists('.gio-banner-alert')).toBe(true);
   });
 
-  it('can be click to close',() => {
+  it('can be click to close', () => {
     const closeMock = jest.fn();
-    const wrapper = mount(
-      <Banner onClose={closeMock}>关闭</Banner>
-    )
+    const wrapper = mount(<Banner onClose={closeMock}>关闭</Banner>);
     wrapper.find('.gio-banner-closeIcon').simulate('click');
     expect(closeMock).toBeCalled();
-  })
+  });
 });

packages/components/src/components/banner/__tests__/__snapshots__/Banner.spec.js.snap

@@ -6,9 +6,7 @@ exports[`Testing Banner should be stable 1`] = `
 >
   <div
     className="gio-banner-content"
-  >
-     
-  </div>
+  />
   <div
     className="gio-banner-button"
   />

packages/components/src/components/list/list.tsx

@@ -7,11 +7,7 @@ import Group from './Group';
 import withGroupedOptions from './utils/withGroupedOptions';
 import { SelectListProps } from './interface';
 
-interface State {
-  value: any | any[];
-}
-
-class SelectList extends React.Component<SelectListProps & ConfigConsumerProps, State> {
+class SelectList extends React.Component<SelectListProps & ConfigConsumerProps> {
   public static defaultProps: Partial<SelectListProps & ConfigConsumerProps> = {
     disabledOptions: [],
     isMultiple: false,
@@ -23,9 +19,6 @@ class SelectList extends React.Component<SelectListProps & ConfigConsumerProps,
   public constructor(props: SelectListProps & ConfigConsumerProps) {
     super(props);
     this.ref = React.createRef();
-    this.state = {
-      value: null,
-    };
   }
 
   private getPopupContainer = () => this.ref.current as HTMLElement;

packages/components/src/components/table/interface.ts

@@ -1,6 +1,7 @@
 import React from 'react';
 import { PaginationProps } from '../pagination/interface';
 import { CheckboxProps } from '../checkbox/interface';
+
 export type AlignType = 'left' | 'center' | 'right';
 export type SortOrder = 'descend' | 'ascend' | null;
 

packages/components/src/components/tree-select/TreeSelect.tsx

@@ -10,7 +10,7 @@ import renderSwitcherIcon from '../tree/iconUtil';
 import SizeContext from '../config-provider/SizeContext';
 import { TreeSelectProps } from './interface';
 
-class TreeSelect<T> extends React.Component<TreeSelectProps<T>, {}> {
+class TreeSelect<T> extends React.Component<TreeSelectProps<T>> {
   public static TreeNode = TreeNode;
 
   public static SHOW_ALL: typeof SHOW_ALL = SHOW_ALL;

packages/components/src/utils/composeRef.ts

@@ -5,6 +5,7 @@ const composeRef = <T>(...refs: React.Ref<T>[]): React.RefCallback<T> => (node:
     if (typeof ref === 'function') {
       ref(node);
     } else if (typeof ref === 'object' && ref && 'current' in ref) {
+      // eslint-disable-next-line no-param-reassign
       (ref as any).current = node;
     }
   });

packages/components/src/utils/hooks/useDebounceLoading.ts

@@ -1,18 +1,19 @@
 import { useState, useCallback, useEffect } from 'react';
 import { debounce } from 'lodash';
 
-const useDebounceLoading = (loading: boolean, delay: number) => {
+const useDebounceLoading = (loading: boolean, delay: number): boolean => {
   const [shouldLoading, setShouldLoading] = useState<boolean>(loading);
+  // eslint-disable-next-line react-hooks/exhaustive-deps
   const debounceFunc = useCallback(
     debounce((_loading: boolean) => setShouldLoading(_loading), delay),
-    [delay],
+    [delay]
   );
   useEffect(() => {
     if (shouldLoading && !loading) {
       debounceFunc.cancel();
     }
     debounceFunc(loading);
-  }, [loading]);
+  }, [debounceFunc, loading, shouldLoading]);
 
   return shouldLoading;
 };

packages/components/src/utils/test/index.ts

@@ -1,3 +1,4 @@
+/* eslint-disable import/prefer-default-export */
 import { act } from 'react-dom/test-utils';
 
 export async function waitForComponentToPaint(wrapper: any, amount = 500) {

packages/components/src/utils/type.ts

@@ -8,6 +8,7 @@ export const tupleNum = <T extends number[]>(...args: T) => args;
  * Extract the type of an element of an array/tuple without performing indexing
  */
 // eslint-disable-next-line no-shadow
+// eslint-disable-next-line @typescript-eslint/no-shadow
 export type ElementOf<T> = T extends (infer E)[] ? E : T extends readonly (infer E)[] ? E : never;
 
 /**