]> git.ipfire.org Git - thirdparty/vuejs/core.git/commitdiff
fix(reactivity): remove Symbol.observable (#968)
authorCédric Exbrayat <cexbrayat@users.noreply.github.com>
Thu, 16 Apr 2020 13:33:30 +0000 (15:33 +0200)
committerGitHub <noreply@github.com>
Thu, 16 Apr 2020 13:33:30 +0000 (09:33 -0400)
* chore: add @types/node as an explicit dependency

As `tsconfig.json` references it, it should be listed in the dependencies.
It currently uses an older version: this commit also bumps to the latest v12 version, and fixes a typing issue.

* fix(reactivity): remove Symbol.observable

`Symbol.observable` is brought by `@types/node@12` and is not a "well-known" typescript symbol https://www.typescriptlang.org/docs/handbook/symbols.html that can be find in lib.es20xx like the others. It has been removed in `@types/node@v13`.
It means that an application using vue@3.0.0-alpha.13 does not compile unless it explicitely adds `@types/node@v12` as a dependency and `node` in its own tsconfig types.

package.json
packages/compiler-sfc/src/templateTransformAssetUrl.ts
packages/reactivity/src/ref.ts
packages/template-explorer/src/index.ts
yarn.lock

index c89aebf0f18ec05f3f9d87e14d19cef7d851516f..39dbfdf47e9bfe33560256930d56e8efca319b0b 100644 (file)
@@ -46,6 +46,7 @@
     "@rollup/plugin-node-resolve": "^7.1.1",
     "@rollup/plugin-replace": "^2.2.1",
     "@types/jest": "^25.1.4",
+    "@types/node": "13.11.1",
     "@types/puppeteer": "^2.0.0",
     "brotli": "^1.3.2",
     "chalk": "^2.4.2",
index 3232c15bb355e7923d4b8b06d8f922c629857f43..b981bf670711d2dfface7af8d1d91ab982885e11 100644 (file)
@@ -69,8 +69,8 @@ export const transformAssetUrl: NodeTransform = (
 }
 
 function getImportsExpressionExp(
-  path: string | undefined,
-  hash: string | undefined,
+  path: string | null,
+  hash: string | null,
   loc: SourceLocation,
   context: TransformContext
 ): ExpressionNode {
index 39d7414a11f4afbba4870ca903c9b27c0721bd93..5b26aec108bbffe2cbbe52de4ecfe38b21a72423 100644 (file)
@@ -151,9 +151,6 @@ type SymbolExtract<T> = (T extends { [Symbol.asyncIterator]: infer V }
   (T extends { [Symbol.iterator]: infer V } ? { [Symbol.iterator]: V } : {}) &
   (T extends { [Symbol.match]: infer V } ? { [Symbol.match]: V } : {}) &
   (T extends { [Symbol.matchAll]: infer V } ? { [Symbol.matchAll]: V } : {}) &
-  (T extends { [Symbol.observable]: infer V }
-    ? { [Symbol.observable]: V }
-    : {}) &
   (T extends { [Symbol.replace]: infer V } ? { [Symbol.replace]: V } : {}) &
   (T extends { [Symbol.search]: infer V } ? { [Symbol.search]: V } : {}) &
   (T extends { [Symbol.species]: infer V } ? { [Symbol.species]: V } : {}) &
index f5579fbd6bf7ce5d6dbb5e17562015af05e7d1c1..e16c1b6257150c6aaddb01e5bb23f314a019bc28 100644 (file)
@@ -97,7 +97,7 @@ window.init = () => {
     }
   }
 
-  const sharedEditorOptions: m.editor.IEditorConstructionOptions = {
+  const sharedEditorOptions: m.editor.IStandaloneEditorConstructionOptions = {
     theme: 'vs-dark',
     fontSize: 14,
     wordWrap: 'on',
index 46783b009b690de444ae5b77a3da257d68fdd66b..8be1b15114bc4e857df10a971452f0a5fe5e428c 100644 (file)
--- a/yarn.lock
+++ b/yarn.lock
   resolved "https://registry.yarnpkg.com/@types/node/-/node-10.17.13.tgz#ccebcdb990bd6139cd16e84c39dc2fb1023ca90c"
   integrity sha512-pMCcqU2zT4TjqYFrWtYHKal7Sl30Ims6ulZ4UFXxI4xbtQqK/qqKwkDoBFCfooRqqmRu9vY3xaJRwxSh673aYg==
 
+"@types/node@13.11.1":
+  version "13.11.1"
+  resolved "https://registry.yarnpkg.com/@types/node/-/node-13.11.1.tgz#49a2a83df9d26daacead30d0ccc8762b128d53c7"
+  integrity sha512-eWQGP3qtxwL8FGneRrC5DwrJLGN4/dH1clNTuLfN81HCrxVtxRjygDTUoZJ5ASlDEeo0ppYFQjQIlXhtXpOn6g==
+
 "@types/prettier@^1.19.0":
   version "1.19.1"
   resolved "https://registry.yarnpkg.com/@types/prettier/-/prettier-1.19.1.tgz#33509849f8e679e4add158959fdb086440e9553f"