is-arguments - npm Package Compare versions

Comparing version 1.0.3 to 1.0.4

CHANGELOG.md

@@ -1,3 +0,7 @@
-1.0.3 / 2015-09-21
+1.0.4 / 2018-11-05
 ==================
+  * [Fix] Fix errors about `in` operator (#22)
+
+1.0.3 / 2018-11-02
+==================
   * [Fix] add awareness of Symbol.toStringTag (#20)
@@ -4,0 +8,0 @@ * [Dev Deps] update `eslint`, `@ljharb/eslint-config`, `tape`, `jscs`, `nsp`

index.js

@@ -7,3 +7,3 @@ 'use strict';
 var isStandardArguments = function isArguments(value) {
-	if (hasToStringTag && value && Symbol.toStringTag in value) {
+	if (hasToStringTag && value && typeof value === 'object' && Symbol.toStringTag in value) {
 		return false;
@@ -10,0 +10,0 @@ }

package.json

 {
 	"name": "is-arguments",
-	"version": "1.0.3",
+	"version": "1.0.4",
 	"description": "Is this an arguments object? It's a harder question than you think.",
@@ -5,0 +5,0 @@ "author": {

test.js

@@ -10,3 +10,4 @@ 'use strict';
 	t.notOk(isArguments({}), 'object is not arguments');
-	t.notOk(isArguments(''), 'string is not arguments');
+	t.notOk(isArguments(''), 'empty string is not arguments');
+	t.notOk(isArguments('foo'), 'string is not arguments');
 	t.notOk(isArguments({ length: 2 }), 'naive array-like is not arguments');
@@ -13,0 +14,0 @@ t.end();

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

Improved metrics

Total package byte prevSize

19750

increased by0.91%

Lines of code

176

increased by0.57%

No dependency changes