Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork40
chore: update deps#1141
base:master
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
chore: update deps#1141
Changes fromall commits
c8be32c
46c146d
71bcffa
eebe547
7432aa7
File filter
Filter by extension
Conversations
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -20,10 +20,10 @@ const loader: loader.Loader = function (source, map) { | ||
if (!registerModules) { | ||
registerModules = defaultMatch; | ||
for (const key in<any>ignoredFiles) { | ||
registerModules = `(?<!${escapeRegExp(ignoredFiles[key])})` + registerModules; | ||
} | ||
registerModules =<any>new RegExp(registerModules); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. Opt for This any casting seems dangerous and seemingly dances around the typings issues that exist here, while functionally identical to before, is it safe? Looks more like the upstream typings are not ideal for this Personally i'd opt for something more typesafe given the typings for letregisterModulesSafe:RegExp;if(!registerModules){registerModules=defaultMatch;// I assume you don't want to use a booleanif(Array.isArray(ignoredFiles)){for(constignoredFileofignoredFiles){registerModules+=`(?<!${escapeRegExp(ignoredFile)})`;}}elseif(typeofignoredFiles==="string"){registerModules+=`(?<!${escapeRegExp(ignoredFiles)})`;}registerModulesSafe=newRegExp(registerModules)asany;} Then you can more safely load this expression further down: }elseif(registerModules||registerModulesSafe){...constcontext=require.context("~/",true,${registerModules||registerModulesSafe});...} | ||
} | ||
if (unitTesting) { | ||
@@ -34,7 +34,7 @@ const loader: loader.Loader = function (source, map) { | ||
const hmr = ` | ||
if (module.hot) { | ||
const hmrUpdate = require("@nativescript/webpack/hmr").hmrUpdate; | ||
global.__coreModulesLiveSync = global.__onLiveSync; | ||
global.__onLiveSync = function () { | ||
@@ -54,7 +54,7 @@ const loader: loader.Loader = function (source, map) { | ||
} | ||
`; | ||
let sourceModule = "@nativescript/core"; | ||
if (platform && platform !== "ios" && platform !== "android") { | ||
sourceModule = `nativescript-platform-${platform}`; | ||
@@ -88,8 +88,8 @@ const loader: loader.Loader = function (source, map) { | ||
source = ` | ||
require("${ | ||
angular ? | ||
'@nativescript/webpack/load-application-css-angular' : | ||
'@nativescript/webpack/load-application-css-regular' | ||
}")(); | ||
${source} | ||
`; | ||
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -4,7 +4,7 @@ import { getOptions, urlToRequest } from "loader-utils"; | ||
const betweenQuotesPattern = /('|")(.*?)\1/; | ||
const unpackUrlPattern = /url\(([^\)]+)\)/; | ||
const inlineLoader = "!@nativescript/webpack/css2json-loader?useForImports!" | ||
const loader: loader.Loader = function (content: string, map) { | ||
const options = getOptions(this) || {}; | ||
@@ -26,7 +26,7 @@ const loader: loader.Loader = function (content: string, map) { | ||
} | ||
}); | ||
const str = JSON.stringify(ast, (k, v) => k === "position" ? undefined : v); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. Is removing | ||
this.callback(null, `${dependencies.join("\n")}module.exports = ${str};`); | ||
} | ||
function getImportRules(ast: Stylesheet): Import[] { | ||
This file was deleted.
Uh oh!
There was an error while loading.Please reload this page.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
// this import should be first in order to load some required settings (like globals and reflect-metadata) | ||
import {platformNativeScript } from "@nativescript/angular"; | ||
import { AppModule } from "./app.module"; | ||
platformNativeScript().bootstrapModule(AppModule); |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -13,23 +13,23 @@ | ||
} | ||
}, | ||
"dependencies": { | ||
"@angular/common": "~10.0.0", | ||
"@angular/compiler": "~10.0.0", | ||
"@angular/core": "~10.0.0", | ||
"@angular/forms": "~10.0.0", | ||
"@angular/platform-browser": "~10.0.0", | ||
"@angular/platform-browser-dynamic": "~10.0.0", | ||
"@angular/router": "~10.0.0", | ||
"@nativescript/angular": "next", | ||
"@nativescript/core": "next", | ||
"nativescript-theme-core": "~1.0.2", | ||
"reflect-metadata": "~0.1.8", | ||
"rxjs": "^6.5.5", | ||
"zone.js": "^0.10.3" | ||
}, | ||
"devDependencies": { | ||
"@angular/compiler-cli": "~10.0.0", | ||
"@ngtools/webpack": "~10.0.0", | ||
"@types/chai": "~4.1.7", | ||
"@types/mocha": "~5.2.5", | ||
"@types/node": "~10.12.18", | ||
@@ -42,12 +42,12 @@ | ||
"chai": "4.2.0", | ||
"mochawesome": "~3.1.2", | ||
"nativescript-dev-appium": "next", | ||
"@nativescript/webpack": "next", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. nit: order | ||
"node-sass": "^4.12.0", | ||
"typescript": "~3.9.0" | ||
}, | ||
"scripts": { | ||
"setup": "npm pack ../../ && npm i -D nativescript-webpack*.tgz", | ||
"e2e": "tsc -p e2e && mocha --opts ../config/mocha.opts --recursive e2e --appiumCapsLocation ../config/appium.capabilities.json", | ||
"compile-tests": "tsc -p e2e --watch" | ||
} | ||
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -15,10 +15,6 @@ | ||
"paths": { | ||
"~/*": [ | ||
"app/*" | ||
] | ||
} | ||
}, | ||
This file was deleted.
Uh oh!
There was an error while loading.Please reload this page.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -14,7 +14,7 @@ | ||
}, | ||
"dependencies": { | ||
"nativescript-theme-core": "~1.0.2", | ||
"@nativescript/core": "~6.5.8" | ||
}, | ||
"devDependencies": { | ||
"@types/chai": "~4.1.7", | ||
@@ -28,11 +28,11 @@ | ||
"mocha": "~5.2.0", | ||
"mochawesome": "~3.1.2", | ||
"nativescript-dev-appium": "next", | ||
"@nativescript/webpack": "next", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others.Learn more. nit: order | ||
"node-sass": "4.12.0" | ||
}, | ||
"scripts": { | ||
"setup": "npm pack ../../ && npm i -D nativescript-webpack*.tgz", | ||
"e2e": "mocha --opts ../config/mocha.opts --recursive e2e --appiumCapsLocation ../config/appium.capabilities.json" | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
import {Observable} from '@nativescript/core'; | ||
export class HelloWorldModel extends Observable { | ||
Uh oh!
There was an error while loading.Please reload this page.