args-to-arr - npm Package Compare versions

Comparing version 0.0.1 to 1.0.0

dist/index.cjs.js

 'use strict';
 
 function toArray(args, start) {
-  start = start || 0;
+
+  if (!(args && "length" in args && typeof args.length === "number")) {
+    throw new TypeError((args + " can't be converted to array."));
+  }
+
+  if (start == null) {
+    start = 0;
+  }
+
+  if (typeof start !== "number") {
+    throw new TypeError((start + " is not a number."));
+  }
+
   var params = [];
+
   for (var index = start, len = args.length; index < len; index++) {
     params[index - start] = args[index];
   }
+
   return params;
+
 }
@@ -11,0 +26,0 @@

dist/index.es.js

 function toArray(args, start) {
-  start = start || 0;
+
+  if (!(args && "length" in args && typeof args.length === "number")) {
+    throw new TypeError((args + " can't be converted to array."));
+  }
+
+  if (start == null) {
+    start = 0;
+  }
+
+  if (typeof start !== "number") {
+    throw new TypeError((start + " is not a number."));
+  }
+
   var params = [];
+
   for (var index = start, len = args.length; index < len; index++) {
     params[index - start] = args[index];
   }
+
   return params;
+
 }
@@ -9,0 +24,0 @@

package.json

 {
   "name": "args-to-arr",
-  "version": "0.0.1",
+  "version": "1.0.0",
   "description": "Arguments to array, the easy way",
@@ -8,7 +8,14 @@ "main": "dist/index.cjs.js",
   "types": "types/index.d.ts",
-  "keywords": [],
+  "keywords": [
+    "arguments",
+    "params",
+    "array",
+    "convert",
+    "slice"
+  ],
   "scripts": {
     "dev": "rollup -cw",
     "build": "rollup -c",
-    "test": "jest"
+    "clean": "rm -rf dist coverage",
+    "test": "jest --colors"
   },
@@ -15,0 +22,0 @@ "files": [

README.md

 # args-to-arr
 
+[![CircleCI](https://circleci.com/gh/manferlo81/args-to-arr.svg?style=svg)](https://circleci.com/gh/manferlo81/args-to-arr) [![Greenkeeper badge](https://badges.greenkeeper.io/manferlo81/args-to-arr.svg)](https://greenkeeper.io/) [![npm](https://img.shields.io/npm/v/args-to-arr.svg)](https://www.npmjs.com/package/args-to-arr) [![codecov](https://codecov.io/gh/manferlo81/args-to-arr/branch/master/graph/badge.svg)](https://codecov.io/gh/manferlo81/args-to-arr) [![GitHub](https://img.shields.io/github/license/manferlo81/args-to-arr.svg)](https://github.com/manferlo81/args-to-arr/blob/master/LICENSE)
+
 Arguments to array, the easy way
@@ -16,3 +18,3 @@
 ```typescript
-toArray<T>(arr: ArrayLike<T>, start?: number): T[];
+toArray<T>(arr: ArrayLike<T>, start: number = 0): T[];
 ```
@@ -41,2 +43,2 @@
 
-MIT License
+[MIT License](https://github.com/manferlo81/args-to-arr/blob/master/LICENSE)

src/index.js

 export default function toArray(args, start) {
-  start = start || 0;
+
+  if (!(args && "length" in args && typeof args.length === "number")) {
+    throw new TypeError(`${args} can't be converted to array.`);
+  }
+
+  if (start == null) {
+    start = 0;
+  }
+
+  if (typeof start !== "number") {
+    throw new TypeError(`${start} is not a number.`);
+  }
+
   const params = [];
+
   for (let index = start, len = args.length; index < len; index++) {
     params[index - start] = args[index];
   }
+
   return params;
+
 }

Fixed alerts

No v1

Quality

Package is not semver >=1. This means it is not stable and does not support ^ ranges.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

7281

increased by57.16%

Lines of code

55

increased by77.42%

Number of low quality alerts

0

decreased by-100%

Number of lines in readme file

43

increased by4.88%

No dependency changes