edge-diagnostics-launch - npm Package Compare versions

Comparing version 1.0.0 to 1.0.1

index.js

@@ -7,27 +7,17 @@ var urlparse = require('url').parse
 
+function launch (url, options, callback) {
+  var args = []
 
-function launch(url, options, callback) {
-  var params = []
-
-  // If URL is present, pass it to adaptor that starts Edge with the URL
-  if(url && url.length) {
+  if (url && url.length) {
     var urlObj = urlparse(url, true)
     delete urlObj.search   // url.format does not want search attribute
     url = urlformat(urlObj)
-    params.push('-l ' + url)
+    args.push('-l ' + url)
   }
-  
-  // Fire up process
-  var architecture = os.arch()
-  var adaptorPath = path.join(__dirname, 'node_modules', 'edge-diagnostics-adapter', 'dist', architecture, 'EdgeDiagnosticsAdapter.exe')
-  var command = adaptorPath
-  var adaptorProcess = child_process.execFile(command, params, callback)
 
-  process.on('exit', function(code) {
-    adaptorProcess.kill()
-  })
+  var command = path.resolve(__dirname, '..', 'node_modules', 'edge-diagnostics-adapter', 'dist', os.arch(), 'EdgeDiagnosticsAdapter.exe')
 
-  return process
+  return child_process.execFile(command, args, callback)
 }
 
-module.exports = launch
+module.exports = launch

package.json

 {
   "name": "edge-diagnostics-launch",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "Launcher for Microsoft Edge Diagnostics Adaptor",
@@ -21,4 +21,4 @@ "main": "index.js",
   "dependencies": {
-    "edge-diagnostics-adapter": "0.0.1"
+    "edge-diagnostics-adapter": "0.1.0"
   }
 }

README.md

 # edge-diagnostics-launch
-Launcher for Microsoft Edge Diagnostics Adaptor to make it simpler to start an instance of the Edge with diagnostics enabled.
+Launcher for Microsoft Edge Diagnostics Adapter to make it simpler to start an instance of the Edge with diagnostics enabled.
 
@@ -4,0 +4,0 @@ ## Installation

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Worsened metrics

Total package byte prevSize

2998

decreased by-8.18%

Lines of code

26

decreased by-21.21%

Dependency changes

• + Addededge-diagnostics-adapter@0.1.0(transitive)

• - Removededge-diagnostics-adapter@0.0.1(transitive)

• Updatededge-diagnostics-adapter@0.1.0