chore: add lint rule for no-console

Author: logaretm

docs/highlight.ts

@@ -59,7 +59,9 @@ export default function highlight() {
         node.value = fragment.querySelector('.shiki-snippet')?.outerHTML;
         node.type = 'html';
       } catch (err) {
+        // eslint-disable-next-line no-console
         console.error(err);
+        // eslint-disable-next-line no-console
         console.log(node.lang);
       }
     }

docs/src/utils/github.ts

@@ -5,6 +5,7 @@ export function fetchStarCount() {
       return json.stargazers_count as number;
     })
     .catch(err => {
+      // eslint-disable-next-line no-console
       console.error(err);
 
       return 0;

eslint.config.js

@@ -1,5 +1,5 @@
 import eslint from '@eslint/js';
-import globals from "globals";
+import globals from 'globals';
 import tseslint from 'typescript-eslint';
 
 export default tseslint.config(
@@ -10,44 +10,57 @@ export default tseslint.config(
       sourceType: 'module',
       globals: {
         ...globals.browser,
-      }
+      },
     },
     rules: {
-      "@typescript-eslint/camelcase": "off",
-      "@typescript-eslint/explicit-function-return-type": "off",
-      "@typescript-eslint/no-use-before-define": "off",
-      "@typescript-eslint/explicit-module-boundary-types": "off",
-      "@typescript-eslint/no-explicit-any": "warn"
+      '@typescript-eslint/camelcase': 'off',
+      '@typescript-eslint/explicit-function-return-type': 'off',
+      '@typescript-eslint/no-use-before-define': 'off',
+      '@typescript-eslint/explicit-module-boundary-types': 'off',
+      '@typescript-eslint/no-explicit-any': 'warn',
+      'no-console': 'error',
     },
   },
   {
-    files: ["scripts/**/*"],
+    files: ['scripts/**/*'],
     languageOptions: {
-      'sourceType': 'commonjs',
+      sourceType: 'commonjs',
       globals: {
-        ...globals.nodeBuiltin
-      }
+        ...globals.nodeBuiltin,
+      },
     },
   },
   {
-    files: [ "docs/scripts/**/*", "docs/*.config.js"],
+    files: ['docs/scripts/**/*', 'docs/*.config.js'],
     languageOptions: {
-      'sourceType': 'commonjs',
+      sourceType: 'commonjs',
       globals: {
-        ...globals.node
-      }
+        ...globals.node,
+      },
+    },
+  },
+  {
+    files: ['scripts/**/*', 'packages/**/*.spec.ts'],
+    languageOptions: {
+      sourceType: 'commonjs',
+      globals: {
+        ...globals.nodeBuiltin,
+      },
+    },
+    rules: {
+      'no-console': 'off',
     },
   },
   {
     ignores: [
-      "packages/vee-validate/dist/*",
-      "packages/yup/dist/*",
-      "packages/zod/dist/*",
-      "packages/valibot/dist/*",
-      "packages/joi/dist/*",
-      "packages/rules/dist/*",
-      "packages/i18n/dist/*",
-      "packages/nuxt/dist/*",
-    ]
-  }
+      'packages/vee-validate/dist/*',
+      'packages/yup/dist/*',
+      'packages/zod/dist/*',
+      'packages/valibot/dist/*',
+      'packages/joi/dist/*',
+      'packages/rules/dist/*',
+      'packages/i18n/dist/*',
+      'packages/nuxt/dist/*',
+    ],
+  },
 );

packages/i18n/src/index.ts

@@ -153,12 +153,14 @@ async function loadLocaleFromURL(url: string) {
     }).then(res => res.json());
 
     if (!locale.code) {
+      // eslint-disable-next-line no-console
       console.error('Could not identify locale, ensure the locale file contains `code` field');
       return;
     }
 
     localize({ [locale.code]: locale });
   } catch (err) {
+    // eslint-disable-next-line no-console
     console.error(`Failed to load locale `);
   }
 }

packages/valibot/src/index.ts

@@ -24,7 +24,7 @@ import {
   StrictObjectSchema,
   LooseObjectSchema,
   getDotPath,
-  Config, 
+  Config,
 } from 'valibot';
 import { isIndex, isObject, merge, normalizeFormPath } from '../../shared';
 
@@ -34,7 +34,7 @@ export function toTypedSchema<
     | BaseSchemaAsync<unknown, unknown, BaseIssue<unknown>>,
   TInferOutput = InferOutput<TSchema>,
   TInferInput = PartialDeep<InferInput<TSchema>>,
->(valibotSchema: TSchema, config?: Config<InferIssue<TSchema>> ): TypedSchema<TInferInput, TInferOutput> {
+>(valibotSchema: TSchema, config?: Config<InferIssue<TSchema>>): TypedSchema<TInferInput, TInferOutput> {
   const schema: TypedSchema = {
     __type: 'VVTypedSchema',
     async parse(value) {
@@ -93,6 +93,7 @@ export function toTypedSchema<
         };
       } catch {
         if (__DEV__) {
+          // eslint-disable-next-line no-console
           console.warn(`Failed to describe path ${path} on the schema, returning a default description.`);
         }
 

packages/vee-validate/src/useField.ts

@@ -552,6 +552,7 @@ function useVModel<TValue = unknown>({ prop, value, handleChange, shouldValidate
   /* istanbul ignore next */
   if (!vm || !prop) {
     if (__DEV__) {
+      // eslint-disable-next-line no-console
       console.warn('Failed to setup model events because `useField` was not called in setup.');
     }
     return;

packages/yup/src/index.ts

@@ -83,6 +83,7 @@ export function toTypedSchema<TSchema extends Schema, TOutput = InferType<TSchem
         return getDescriptionFromYupSpec(description);
       } catch {
         if (__DEV__) {
+          // eslint-disable-next-line no-console
           console.warn(`Failed to describe path ${path} on the schema, returning a default description.`);
         }
 

packages/zod/src/index.ts

@@ -77,6 +77,7 @@ export function toTypedSchema<
         };
       } catch {
         if (__DEV__) {
+          // eslint-disable-next-line no-console
           console.warn(`Failed to describe path ${path} on the schema, returning a default description.`);
         }