fix: oxlint no-console rule

DevoInc · Jul 11, 2024 · d797c63 · d797c63
1 parent 8585936
commit d797c63
Show file tree

Hide file tree

Showing 10 changed files with 24 additions and 6 deletions.
diff --git a/dot.mjs b/dot.mjs
@@ -13,6 +13,7 @@ const input = readFileSync('/dev/stdin').toString();
     writeFileSync('/dev/stdout', Buffer.from(output));
   } catch (error) {
     graph = new Viz({ Module, render });
+    // eslint-disable-next-line no-console
     console.error(error);
   }
 })();
diff --git a/packages/code/src/components/DiffEditor/stories/cases/Actions.tsx b/packages/code/src/components/DiffEditor/stories/cases/Actions.tsx
@@ -26,6 +26,7 @@ export const Actions = ({ ...props }: Partial<DiffEditorProps>) => {
         console.log('Copied to clipboard:', value);
       })
       .catch((err) => {
+        // eslint-disable-next-line no-console
         console.error('Failed to copy to clipboard', err);
       });
   };
@@ -38,7 +39,10 @@ export const Actions = ({ ...props }: Partial<DiffEditorProps>) => {
         const currentEditor = editorRef.current.getModifiedEditor();
         currentEditor.setValue(currentEditor.getValue().concat(value));
       })
-      .catch((err) => console.error('Failed to paste from clipboard', err));
+      .catch((err) => {
+        // eslint-disable-next-line no-console
+        console.error('Failed to paste from clipboard', err);
+      });
   };
 
   return (

diff --git a/packages/code/src/components/Editor/Editor.stories.tsx b/packages/code/src/components/Editor/Editor.stories.tsx
@@ -205,6 +205,7 @@ export const WithActions: Story = {
             console.log('Copied to clipboard:', value);
           })
           .catch((err) => {
+            // eslint-disable-next-line no-console
             console.error('Failed to copy to clipboard', err);
           });
       };
@@ -218,6 +219,7 @@ export const WithActions: Story = {
             editorRef.current.setValue(currentValue.concat(value));
           })
           .catch((err) => {
+            // eslint-disable-next-line no-console
             console.error('Failed to paste from clipboard', err);
           });
       };

diff --git a/packages/core/src/components/Banner/Banner.stories.tsx b/packages/core/src/components/Banner/Banner.stories.tsx
@@ -38,7 +38,12 @@ export const Subtle: Story = {
     status: 'success',
     content: 'The comment was created successfully.',
     actions: (
-      <Banner.SubtleAction onClick={() => console.info('Clicked!')}>
+      <Banner.SubtleAction
+        onClick={() => {
+          // eslint-disable-next-line no-console
+          console.info('Clicked!');
+        }}
+      >
         Undo
       </Banner.SubtleAction>
     ),

diff --git a/packages/core/src/components/Tabs/Tabs.stories.tsx b/packages/core/src/components/Tabs/Tabs.stories.tsx
@@ -115,6 +115,7 @@ export const Closable: Story = {
       ]);
       useTabsAccessibility({ activeTab, tabsRef });
       //useClosableTabs({ activeTab, tabsRef, tabsLength: tabs.length });
+      // eslint-disable-next-line no-console
       console.log(activeTab);
       return (
         <Tabs aria-label="Demo tabs" contained>

diff --git a/packages/core/src/components/Tabs/hooks/useClosableTabs.ts b/packages/core/src/components/Tabs/hooks/useClosableTabs.ts
@@ -12,11 +12,9 @@ export const useClosableTabs = ({
 }) => {
   const itemsRef = React.useRef<NodeListOf<Element>>([]);
   React.useEffect(() => {
-    console.log(itemsRef.current);
     const removedItemIndex = Array.from(itemsRef.current).findIndex(
       (item) => !document.body.contains(item),
     );
-    console.log(removedItemIndex);
     itemsRef.current = tabsRef.current.querySelectorAll('[aria-selected]');
     if (activeTab > itemsRef.current.length - 1) {
       getEnabledItemByIndex({ items: itemsRef.current, index: 'last' });

diff --git a/packages/core/src/components/Tabs/hooks/useTabsAccessibility.ts b/packages/core/src/components/Tabs/hooks/useTabsAccessibility.ts
@@ -34,13 +34,11 @@ export const useTabsAccessibility = ({
     'rightarrow',
     () => {
       const items = tabsRef.current.querySelectorAll('[aria-selected]');
-      console.log(items);
       const nextEnabledTabItem = getEnabledItemByIncrement({
         items,
         currentItemIndex: activeTab,
         inc: 1,
       });
-      console.log(nextEnabledTabItem);
       nextEnabledTabItem.focus();
       nextEnabledTabItem.click();
     },

diff --git a/packages/datetime/src/components/DateTimeFloatingPicker/DateTimeFloatingPicker.stories.tsx b/packages/datetime/src/components/DateTimeFloatingPicker/DateTimeFloatingPicker.stories.tsx
@@ -28,6 +28,7 @@ export const Base: Story = {
           value={value}
           onChange={(ts) => {
             setValue(ts);
+            // eslint-disable-next-line no-console
             console.log({ onChange: ts });
           }}
         />
@@ -52,6 +53,7 @@ export const WithoutButtons: Story = {
           autoApply={true}
           onChange={(ts) => {
             setValue(ts);
+            // eslint-disable-next-line no-console
             console.log(`OnChange ${ts}`);
           }}
         />
@@ -86,6 +88,7 @@ export const CustomParser: Story = {
           }}
           onChange={(ts) => {
             setValue(ts);
+            // eslint-disable-next-line no-console
             console.log(`OnChange ${ts}`);
           }}
         />

diff --git a/...tetime/src/components/DateTimeRangeFloatingPicker/DateTimeRangeFloatingPicker.stories.tsx b/...tetime/src/components/DateTimeRangeFloatingPicker/DateTimeRangeFloatingPicker.stories.tsx
@@ -35,6 +35,7 @@ export const Base: Story = {
           value={value}
           onChange={(newValue) => {
             setValue(newValue);
+            // eslint-disable-next-line no-console
             console.log('Has changed', newValue);
           }}
         />
@@ -53,9 +54,11 @@ export const DefaultPresets: Story = {
           value={date}
           onApply={onApply(setDate)}
           onChange={(range) => {
+            // eslint-disable-next-line no-console
             console.log('something has changed', range);
           }}
           onRealTimeClick={(event) => {
+            // eslint-disable-next-line no-console
             console.log('RT button clicked', event);
           }}
           presets={defaultPresets}

diff --git a/packages/table/stories/BasicTable.stories.tsx b/packages/table/stories/BasicTable.stories.tsx
@@ -57,6 +57,7 @@ export const Base: Story = {
                   <Menu.Item
                     label="Option 1"
                     onClick={() => {
+                      // eslint-disable-next-line no-console
                       console.log(colDef);
                     }}
                   />
@@ -163,12 +164,14 @@ export const Base: Story = {
             {
               Icon: GIEyeViewFilled,
               onClick: (rowIndex) => {
+                // eslint-disable-next-line no-console
                 console.log(`Action 1 on row ${rowIndex}`);
               },
             },
             {
               Icon: GIPencilEdit,
               onClick: (rowIndex) => {
+                // eslint-disable-next-line no-console
                 console.log(`Action 2 on row ${rowIndex}`);
               },
             },