Skip to content

Commit

Permalink
fix: remove assert due to safari bug (#798)
Browse files Browse the repository at this point in the history
* fix: remove assert due to safari bug

* fix: bump peer dependencies

* fix: lint issue
  • Loading branch information
diervo authored Nov 2, 2018
1 parent 7076bd9 commit 9381315
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 8 deletions.
5 changes: 2 additions & 3 deletions packages/lwc-engine/src/framework/def.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,13 +13,11 @@ import {
freeze,
create,
ArrayIndexOf,
toString,
ArrayPush,
defineProperty,
getOwnPropertyDescriptor,
getOwnPropertyNames,
getPrototypeOf,
isString,
isFunction,
isNull,
setPrototypeOf,
Expand Down Expand Up @@ -127,7 +125,8 @@ function createComponentDef(Ctor: ComponentConstructor): ComponentDef {

// local to dev block
const ctorName = Ctor.name;
assert.isTrue(ctorName && isString(ctorName), `${toString(Ctor)} should have a "name" property with string value, but found ${ctorName}.`);
// Removing the following assert until https://bugs.webkit.org/show_bug.cgi?id=190140 is fixed.
// assert.isTrue(ctorName && isString(ctorName), `${toString(Ctor)} should have a "name" property with string value, but found ${ctorName}.`);
assert.isTrue(Ctor.constructor, `Missing ${ctorName}.constructor, ${ctorName} should have a "constructor" property.`);
}

Expand Down
6 changes: 3 additions & 3 deletions packages/lwc-jest-transformer/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,9 @@
"deasync": "0.1.12"
},
"peerDependencies": {
"lwc-compiler": "0.30.x",
"lwc-engine": "0.30.x",
"lwc-errors": "0.30.x"
"lwc-compiler": "0.31.x",
"lwc-engine": "0.31.x",
"lwc-errors": "0.31.x"
},
"devDependencies": {
"lwc-compiler": "0.31.1",
Expand Down
4 changes: 2 additions & 2 deletions packages/rollup-plugin-lwc-compiler/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
"rollup-pluginutils": "^2.0.1"
},
"peerDependencies": {
"lwc-compiler": "0.30.x",
"lwc-engine": "0.30.x"
"lwc-compiler": "0.31.x",
"lwc-engine": "0.31.x"
}
}

0 comments on commit 9381315

Please sign in to comment.