Selaa lähdekoodia

chore: remove error expectations that no longer apply

There were a few @ts-expect-error and eslint-disable lines caused by typings
which have been now fixed in the underlying libraries and so we can now remove
those annotatinos.
Connor Prussin 8 kuukautta sitten
vanhempi
sitoutus
c3e373ebf2

+ 0 - 3
apps/staking/src/components/ErrorMessage/index.tsx

@@ -49,9 +49,6 @@ const UnknownError = ({ error }: { error: unknown }) => {
         </div>
       </Button>
       <m.div
-        // @ts-expect-error the framer-motion types don't currently expose props
-        // like `className` correctly for some reason, even though this works
-        // correctly...
         className="overflow-hidden pt-1 opacity-60"
         initial={{ height: 0 }}
         animate={{ height: detailsOpen ? "auto" : 0 }}

+ 0 - 3
apps/staking/src/components/Faq/index.tsx

@@ -56,9 +56,6 @@ export const Faq = ({ title, questions, className, ...props }: Props) => {
                 </Button>
               </dt>
               <m.dt
-                // @ts-expect-error the framer-motion types don't currently
-                // expose props like `className` correctly for some reason, even
-                // though this works correctly...
                 className="-mt-1 flex max-w-prose flex-col gap-4 overflow-hidden font-light"
                 initial={{ height: openItem === i ? "auto" : 0 }}
                 animate={{ height: openItem === i ? "auto" : 0 }}

+ 0 - 5
packages/component-library/src/Breadcrumbs/index.tsx

@@ -41,11 +41,6 @@ export const Breadcrumbs = ({ label, className, items, ...props }: Props) => (
                 <Button
                   size="xs"
                   variant="outline"
-                  // I'm not quite sure why this is triggering, I'll need to
-                  // figure this out later.  Something in Phosphor's types is
-                  // incorrect and is making eslint think this icon is an error
-                  // object somehow...
-                  // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
                   beforeIcon={House}
                   hideText
                   href="/"

+ 0 - 1
packages/component-library/src/Drawer/index.tsx

@@ -124,7 +124,6 @@ export const Drawer = ({
           if (e.type !== "pointercancel" && velocity.y > 10) {
             state.close();
           } else {
-            // eslint-disable-next-line @typescript-eslint/no-floating-promises
             animate(y, "0", {
               type: "inertia",
               bounceStiffness: 300,

+ 0 - 8
packages/component-library/src/Paginator/index.tsx

@@ -107,10 +107,6 @@ const PaginatorToolbar = ({
     <Toolbar aria-label="Page" className={styles.paginatorToolbar ?? ""}>
       <PageSelector
         hideText
-        // I'm not quite sure why this is triggering, I'll need to figure this
-        // out later.  Something in Phosphor's types is incorrect and is making
-        // eslint think this icon is an error object somehow...
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
         beforeIcon={CaretLeft}
         isDisabled={currentPage === 1}
         page={1}
@@ -136,10 +132,6 @@ const PaginatorToolbar = ({
       })}
       <PageSelector
         hideText
-        // I'm not quite sure why this is triggering, I'll need to figure this
-        // out later.  Something in Phosphor's types is incorrect and is making
-        // eslint think this icon is an error object somehow...
-        // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
         beforeIcon={CaretRight}
         isDisabled={currentPage === numPages}
         page={numPages}