Merge pull request #10 from riitasointi/linter-rules

Remove unused variables
vrk-kpa · Mar 21, 2023 · 57a4f33 · 57a4f33
2 parents b876eee + 7e42280
commit 57a4f33
Show file tree

Hide file tree

Showing 17 changed files with 11 additions and 66 deletions.
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -7,7 +7,6 @@
   ],
   "plugins": ["@typescript-eslint", "prettier"],
   "rules": {
-    "@typescript-eslint/no-unused-vars": "off",
     "@typescript-eslint/no-explicit-any": "off",
     "@typescript-eslint/no-unnecessary-type-constraint": "off",
     "@typescript-eslint/ban-types": ["off", { "types": ["{}", "object"] }],

diff --git a/components/ComponentExample/ComponentExample.tsx b/components/ComponentExample/ComponentExample.tsx
@@ -8,7 +8,6 @@ import MobileDevice from '../MobileDevice/MobileDevice';
 import ShowcaseBox from '../ShowcaseBox/ShowcaseBox';
 import React from 'react';
 import ComponentCode from './ComponentCode';
-import styles from './ComponentExample.module.scss';
 
 type ComponentExampleVariant = 'normal' | 'mobile_device';
 
@@ -18,7 +17,6 @@ interface ComponentExampleProps {
   children?: ReactNode;
   codeString?: string;
   style?: CSSProperties;
-  noCode?: boolean;
 }
 
 const getWithoutWrappers = (children: any): ReactNode[] =>
@@ -36,7 +34,6 @@ const ComponentExample: React.FunctionComponent<ComponentExampleProps> = ({
   filterPropsInExample,
   codeString,
   style,
-  noCode,
 }) => {
   const showcase =
     variant === 'mobile_device' ? (

diff --git a/components/Footer/Footer.tsx b/components/Footer/Footer.tsx
@@ -8,7 +8,6 @@ import {
   Link as SuomifiLink,
   Text,
 } from 'suomifi-ui-components';
-import GitHubLogo from '../../assets/icons/github.svg';
 import styles from './Footer.module.scss';
 
 const Footer: React.FunctionComponent = () => {

diff --git a/components/SpacingShowcase/SpacingShowcase.tsx b/components/SpacingShowcase/SpacingShowcase.tsx
@@ -19,9 +19,7 @@ export const cssValueToString = (cssValue: number | string | ValueUnit) => {
   return typeof cssValue === 'number' ? cssValue.toString(10) : cssValue;
 };
 
-const Container = styled(({ size, name, ...passProps }) => (
-  <div {...passProps} />
-))(
+const Container = styled(({ ...passProps }) => <div {...passProps} />)(
   ({ size }) => `
   min-height: calc(${size} + 2em);
   max-width: 420px;

diff --git a/layouts/InfopageLayout/InfopageLayout.tsx b/layouts/InfopageLayout/InfopageLayout.tsx
@@ -1,4 +1,3 @@
-import { useTranslation } from 'next-export-i18n';
 import { ReactNode } from 'react';
 import { Block } from 'suomifi-ui-components';
 import styles from './InfopageLayout.module.scss';
@@ -10,7 +9,6 @@ interface InfopageLayoutProps {
 const InfopageLayout: React.FunctionComponent<InfopageLayoutProps> = ({
   children,
 }: InfopageLayoutProps) => {
-  const { t } = useTranslation();
   return (
     <>
       <Block className="container">

diff --git a/layouts/PlainLayout/PlainLayout.tsx b/layouts/PlainLayout/PlainLayout.tsx
@@ -1,4 +1,3 @@
-import { useTranslation } from 'next-export-i18n';
 import { ReactNode } from 'react';
 import { Block } from 'suomifi-ui-components';
 
@@ -9,7 +8,6 @@ interface PlainLayoutProps {
 const PlainLayout: React.FunctionComponent<PlainLayoutProps> = ({
   children,
 }: PlainLayoutProps) => {
-  const { t } = useTranslation();
   return (
     <>
       <Block className="container">

diff --git a/pages/components/button.tsx b/pages/components/button.tsx
@@ -239,7 +239,6 @@ const ButtonPage: NextPage = () => {
           </Paragraph>
 
           <ComponentExample
-            noCode
             style={{
               padding: defaultSuomifiTheme.spacing.m,
               marginBottom: defaultSuomifiTheme.spacing.s,
@@ -265,7 +264,6 @@ const ButtonPage: NextPage = () => {
             </Block>
           </ComponentExample>
           <ComponentExample
-            noCode
             style={{
               padding: defaultSuomifiTheme.spacing.m,
               marginBottom: defaultSuomifiTheme.spacing.s,
@@ -295,7 +293,6 @@ const ButtonPage: NextPage = () => {
             </Block>
           </ComponentExample>
           <ComponentExample
-            noCode
             style={{
               padding: defaultSuomifiTheme.spacing.m,
               marginBottom: defaultSuomifiTheme.spacing.s,
@@ -325,7 +322,6 @@ const ButtonPage: NextPage = () => {
             </Block>
           </ComponentExample>
           <ComponentExample
-            noCode
             style={{
               padding: defaultSuomifiTheme.spacing.m,
               marginBottom: defaultSuomifiTheme.spacing.s,
@@ -359,7 +355,6 @@ const ButtonPage: NextPage = () => {
             </Block>
           </ComponentExample>
           <ComponentExample
-            noCode
             style={{
               padding: defaultSuomifiTheme.spacing.m,
               background: defaultSuomifiTheme.colors.whiteBase,

diff --git a/pages/components/loadingspinner.tsx b/pages/components/loadingspinner.tsx
@@ -152,9 +152,7 @@ const Page: NextPage = () => {
   );
 };
 
-const BasicExample = ({
-  ...passProps
-}: Partial<LoadingSpinnerProps>): JSX.Element => {
+const BasicExample = ({}: Partial<LoadingSpinnerProps>): JSX.Element => {
   const { t } = useTranslation();
   const [visible, setVisible] = useState(false);
   const [status, setStatus] = useState<LoadingSpinnerStatus>('loading');
@@ -175,7 +173,6 @@ const BasicExample = ({
 
   return (
     <ComponentExample
-      noCode
       style={{
         marginBottom: defaultSuomifiTheme.spacing.s,
         display: 'flex',
@@ -224,9 +221,7 @@ const BasicExample = ({
   );
 };
 
-const SuccessExample = ({
-  ...passProps
-}: Partial<LoadingSpinnerProps>): JSX.Element => {
+const SuccessExample = ({}: Partial<LoadingSpinnerProps>): JSX.Element => {
   const { t } = useTranslation();
   const [visible, setVisible] = useState(false);
   const [status, setStatus] = useState<LoadingSpinnerStatus>('failed');
@@ -247,7 +242,6 @@ const SuccessExample = ({
 
   return (
     <ComponentExample
-      noCode
       style={{
         marginBottom: defaultSuomifiTheme.spacing.s,
         display: 'flex',
@@ -299,9 +293,7 @@ const SuccessExample = ({
   );
 };
 
-const FailingExample = ({
-  ...passProps
-}: Partial<LoadingSpinnerProps>): JSX.Element => {
+const FailingExample = ({}: Partial<LoadingSpinnerProps>): JSX.Element => {
   const { t } = useTranslation();
   const [visible, setVisible] = useState(false);
   const [status, setStatus] = useState<LoadingSpinnerStatus>('success');
@@ -322,7 +314,6 @@ const FailingExample = ({
 
   return (
     <ComponentExample
-      noCode
       style={{
         marginBottom: defaultSuomifiTheme.spacing.s,
         display: 'flex',

diff --git a/pages/components/modal.tsx b/pages/components/modal.tsx
@@ -53,7 +53,7 @@ const Page: NextPage = () => {
         </Paragraph>
 
         <Block variant="section">
-          <ConfirmExample title="" desc="" noCode />
+          <ConfirmExample />
         </Block>
 
         <Block variant="section" className="my-xl">
@@ -99,7 +99,7 @@ const Page: NextPage = () => {
           </Paragraph>
         </Block>
 
-        <ConfirmExample title="" desc="" noCode={false} />
+        <ConfirmExample />
 
         <Block variant="section">
           <Heading variant="h2" className="my-xl">
@@ -268,16 +268,7 @@ const DefaultExample = ({ ...passProps }: Partial<ModalProps>): JSX.Element => {
   );
 };
 
-const ConfirmExample = ({
-  title,
-  desc,
-  noCode,
-  ...passProps
-}: Partial<ModalProps> & {
-  title: string;
-  desc: string;
-  noCode: boolean;
-}): JSX.Element => {
+const ConfirmExample = ({ ...passProps }: Partial<ModalProps>): JSX.Element => {
   const { t } = useTranslation();
   const [visible, setVisible] = useState(false);
   return (

diff --git a/pages/components/multiselect.tsx b/pages/components/multiselect.tsx
@@ -5,7 +5,6 @@ import SideNavLayout from '../../layouts/SideNavLayout/SideNavLayout';
 import { navItems } from '../../utils/components-sidenav';
 import InfoBox from '../../components/InfoBox/InfoBox';
 import ComponentExample from '../../components/ComponentExample/ComponentExample';
-import { useState } from 'react';
 import {
   Block,
   Heading,

diff --git a/pages/components/notification.tsx b/pages/components/notification.tsx
@@ -5,7 +5,6 @@ import SideNavLayout from '../../layouts/SideNavLayout/SideNavLayout';
 import { navItems } from '../../utils/components-sidenav';
 import InfoBox from '../../components/InfoBox/InfoBox';
 import ComponentExample from '../../components/ComponentExample/ComponentExample';
-import { useState } from 'react';
 import {
   Block,
   Heading,

diff --git a/pages/components/pagination.tsx b/pages/components/pagination.tsx
@@ -113,9 +113,7 @@ const Page: NextPage = () => {
   );
 };
 
-const FirstExample = ({
-  ...passProps
-}: Partial<PaginationProps> & {}): JSX.Element => {
+const FirstExample = ({}: Partial<PaginationProps> & {}): JSX.Element => {
   const [current, setCurrent] = useState(2);
   const lastPage = 8;
 
@@ -161,9 +159,7 @@ const FirstExample = ({
   );
 };
 
-const BasicExample = ({
-  ...passProps
-}: Partial<PaginationProps> & {}): JSX.Element => {
+const BasicExample = ({}: Partial<PaginationProps> & {}): JSX.Element => {
   const arrLength = 100;
   const step = 5;
 
@@ -231,9 +227,7 @@ const BasicExample = ({
   );
 };
 
-const SuccessExample = ({
-  ...passProps
-}: Partial<PaginationProps> & {}): JSX.Element => {
+const SuccessExample = ({}: Partial<PaginationProps> & {}): JSX.Element => {
   const [current, setCurrent] = useState(2);
   const lastPage = 8;
 
@@ -269,9 +263,7 @@ const SuccessExample = ({
   );
 };
 
-const FailingExample = ({
-  ...passProps
-}: Partial<PaginationProps> & {}): JSX.Element => {
+const FailingExample = ({}: Partial<PaginationProps> & {}): JSX.Element => {
   const [current, setCurrent] = useState(2);
   const lastPage = 8;
 

diff --git a/pages/components/radiobutton.tsx b/pages/components/radiobutton.tsx
@@ -11,8 +11,6 @@ import {
   Text,
   Paragraph,
   ExternalLink,
-  Link as ExampleLink,
-  ExternalLink as ExampleExternalLink,
   defaultSuomifiTheme,
   RadioButton,
   RadioButtonGroup,

diff --git a/pages/components/servicenavigation.tsx b/pages/components/servicenavigation.tsx
@@ -11,8 +11,6 @@ import {
   Text,
   Paragraph,
   ExternalLink,
-  Link as ExampleLink,
-  ExternalLink as ExampleExternalLink,
   ServiceNavigation,
   ServiceNavigationItem,
   RouterLink,

diff --git a/pages/components/sidenavigation.tsx b/pages/components/sidenavigation.tsx
@@ -11,8 +11,6 @@ import {
   Text,
   Paragraph,
   ExternalLink,
-  Link as ExampleLink,
-  ExternalLink as ExampleExternalLink,
   SideNavigation,
   SideNavigationItem,
   RouterLink,

diff --git a/pages/components/text.tsx b/pages/components/text.tsx
@@ -3,16 +3,13 @@ import Head from 'next/head';
 import { useTranslation } from 'next-export-i18n';
 import SideNavLayout from '../../layouts/SideNavLayout/SideNavLayout';
 import { navItems } from '../../utils/components-sidenav';
-import InfoBox from '../../components/InfoBox/InfoBox';
 import ComponentExample from '../../components/ComponentExample/ComponentExample';
 import {
   Block,
   Heading,
   Text,
   Paragraph,
   ExternalLink,
-  Link as ExampleLink,
-  ExternalLink as ExampleExternalLink,
 } from 'suomifi-ui-components';
 
 const Page: NextPage = () => {

diff --git a/pages/patterns/multi-insert.tsx b/pages/patterns/multi-insert.tsx
@@ -6,13 +6,11 @@ import {
   Text,
   Paragraph,
   Block,
-  RouterLink,
   ExternalLink,
 } from 'suomifi-ui-components';
 import SideNavLayout from '../../layouts/SideNavLayout/SideNavLayout';
 import { navItems } from '../../utils/patterns-sidenav';
 import InfoBox from '../../components/InfoBox/InfoBox';
-import Link from 'next/link';
 import { MultiInsert } from '../../components/MultiInsert/MultiInsert';
 
 const MultiInsertPage: NextPage = () => {