fix: properly declare optional peer deps

We used `optionalDependencies` in vuejs#1958 but I think the proper way is to use `peerDependenciesMeta`. This fixes the current issue where pnpm installs compiler-dom and server-renderer in v3.2.47, despite v3.3.1 being available.
cexbrayat · May 11, 2023 · 0a07785 · 0a07785
1 parent 41b772e
commit 0a07785
Show file tree

Hide file tree

Showing 2 changed files with 39 additions and 75 deletions.
diff --git a/package.json b/package.json
@@ -70,9 +70,13 @@
     "@vue/server-renderer": "^3.0.1",
     "vue": "^3.0.1"
   },
-  "optionalDependencies": {
-    "@vue/compiler-dom": "^3.0.1",
-    "@vue/server-renderer": "^3.0.1"
+  "peerDependenciesMeta": {
+    "@vue/compiler-dom": {
+      "optional": true
+    },
+    "@vue/server-renderer": {
+      "optional": true
+    }
   },
   "author": {
     "name": "Lachlan Miller",
@@ -121,4 +125,4 @@
       ]
     }
   }
-}
+}
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml