build(biome): enable noConsoleLog rule

gitify-app · Jun 18, 2024 · 91afaef · 91afaef
1 parent 83f4da3
commit 91afaef
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 0 deletions.
diff --git a/biome.json b/biome.json
@@ -7,6 +7,9 @@
     "enabled": true,
     "rules": {
       "recommended": true,
+      "suspicious": {
+        "noConsoleLog": "error"
+      },
       "nursery": {
         "noUnusedFunctionParameters": "error",
         "useDefaultSwitchClause": "error"

diff --git a/scripts/notarize.js b/scripts/notarize.js
@@ -10,12 +10,14 @@ const notarizeApp = async (context) => {
   const shouldNotarize = process.env.NOTARIZE === 'true';
 
   if (!shouldNotarize || !isMacOS) {
+    // biome-ignore lint/suspicious/noConsoleLog: console.log is used for debugging
     console.log(
       '  • notarizing      either should not notarize or not building for macOS',
     );
     return;
   }
 
+  // biome-ignore lint/suspicious/noConsoleLog: console.log is used for debugging
   console.log('  • notarizing      started');
 
   return await notarize({

diff --git a/src/utils/auth/migration.ts b/src/utils/auth/migration.ts
@@ -12,6 +12,7 @@ export async function migrateAuthenticatedAccounts() {
   const existing = loadState();
 
   if (hasAccountsToMigrate(existing.auth)) {
+    // biome-ignore lint/suspicious/noConsoleLog: console.log is used for debugging
     console.log('Commencing authenticated accounts migration');
 
     const migratedAccounts = await convertAccounts(existing.auth);
@@ -20,6 +21,7 @@ export async function migrateAuthenticatedAccounts() {
       auth: { ...existing.auth, accounts: migratedAccounts },
       settings: existing.settings,
     });
+    // biome-ignore lint/suspicious/noConsoleLog: console.log is used for debugging
     console.log('Authenticated accounts migration complete');
   }
 }