Update to rome stable.

Includes workarounds for: - rome/tools#3734 - rome/tools#3735
github · Nov 15, 2022 · a4a1128 · a4a1128
1 parent 8e72298
commit a4a1128
Show file tree

Hide file tree

Showing 4 changed files with 64 additions and 112 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -36,7 +36,7 @@
     "esbuild": "^0.15.13",
     "jest": "^29.2.2",
     "jest-environment-jsdom": "^29.2.2",
-    "rome": "^0.10.1-next",
+    "rome": "^10.0.1",
     "ts-jest": "^29.0.3",
     "typescript": "^4.8.4"
   },

diff --git a/rome.json b/rome.json
@@ -6,10 +6,7 @@
   "linter": {
     "enabled": true,
     "rules": {
-      "recommended": true,
-      "correctness": {
-        "useSimplifiedLogicExpression": "off"
-      }
+      "recommended": true
     }
   }
 }
diff --git a/src/webauthn-json/basic/supported.ts b/src/webauthn-json/basic/supported.ts
@@ -4,9 +4,12 @@
 // https://developers.google.com/web/updates/2018/03/webauthn-credential-management
 
 export function supported(): boolean {
+  // rome-ignore format: Work around https://github.com/rome/tools/issues/3734
   return !!(
-    navigator.credentials?.create &&
+    // rome-ignore lint(style/useOptionalChain): Optional chaining creates more complicated ES2019 code
+    navigator.credentials &&
+    navigator.credentials.create &&
     navigator.credentials.get &&
-    globalThis.PublicKeyCredential
+    window.PublicKeyCredential
   );
 }