Update copyright dates, restore old Function checking behaviour

Author: michaelkourlas

.github/workflows/node.js.yml

@@ -4,26 +4,25 @@
 name: Node.js CI
 
 on:
-  push:
-    branches: [ master ]
-  pull_request:
-    branches: [ master ]
+    push:
+        branches: [master]
+    pull_request:
+        branches: [master]
 
 jobs:
-  build:
+    build:
+        runs-on: ubuntu-latest
 
-    runs-on: ubuntu-latest
+        strategy:
+            matrix:
+                node-version: [10.x, 12.x, 14.x, 15.x]
+                # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
 
-    strategy:
-      matrix:
-        node-version: [10.x, 12.x, 14.x, 15.x]
-        # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
-
-    steps:
-    - uses: actions/checkout@v2
-    - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v1
-      with:
-        node-version: ${{ matrix.node-version }}
-    - run: npm ci
-    - run: npm run-script build
+        steps:
+            - uses: actions/checkout@v2
+            - name: Use Node.js ${{ matrix.node-version }}
+              uses: actions/setup-node@v1
+              with:
+                  node-version: ${{ matrix.node-version }}
+            - run: npm ci
+            - run: npm run-script build

src/main.ts

@@ -1,5 +1,5 @@
 /**
- * Copyright (C) 2016-2019 Michael Kourlas
+ * Copyright (C) 2016-2020 Michael Kourlas
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

src/options.ts

@@ -1,5 +1,5 @@
 /**
- * Copyright (C) 2016-2019 Michael Kourlas
+ * Copyright (C) 2016-2020 Michael Kourlas
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

src/utils.ts

@@ -1,5 +1,5 @@
 /**
- * Copyright (C) 2016-2019 Michael Kourlas
+ * Copyright (C) 2016-2020 Michael Kourlas
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -30,6 +30,11 @@ export function isArray(val: unknown): val is unknown[] {
     return Object.prototype.toString.call(val) === "[object Array]";
 }
 
+// eslint-disable-next-line @typescript-eslint/ban-types
+export function isFunction(val: unknown): val is Function {
+    return Object.prototype.toString.call(val) === "[object Function]";
+}
+
 export function isSet(val: unknown): val is Set<unknown> {
     return Object.prototype.toString.call(val) === "[object Set]";
 }
@@ -51,7 +56,7 @@ export function isMap(val: unknown): val is Map<unknown, unknown> {
 // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/explicit-module-boundary-types
 export function stringify(value: any): string {
     if (!isUndefined(value) && !isNull(value)) {
-        if (value?.toString) {
+        if (isFunction(value?.toString)) {
             value = value.toString();
         }
     }

test/src/main.ts

@@ -1,5 +1,5 @@
 /**
- * Copyright (C) 2016-2019 Michael Kourlas
+ * Copyright (C) 2016-2020 Michael Kourlas
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

test/src/options.ts

@@ -1,5 +1,5 @@
 /**
- * Copyright (C) 2016-2019 Michael Kourlas
+ * Copyright (C) 2016-2020 Michael Kourlas
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

test/src/utils.ts

@@ -1,5 +1,5 @@
 /**
- * Copyright (C) 2016-2019 Michael Kourlas
+ * Copyright (C) 2016-2020 Michael Kourlas
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -20,6 +20,7 @@ import {
     isMap,
     isNull,
     isObject,
+    isFunction,
     isSet,
     isUndefined,
     stringify,
@@ -83,6 +84,21 @@ describe("utils", () => {
         });
     });
 
+    describe("#isFunction", () => {
+        it("should return true for functions", () => {
+            assert.isTrue(isFunction(() => 0));
+            assert.isTrue(isFunction(() => "test"));
+        });
+
+        it("should return false for values that are not functions", () => {
+            assert.isFalse(isFunction("test"));
+            assert.isFalse(isFunction(3));
+            assert.isFalse(isFunction(undefined));
+            assert.isFalse(isFunction(true));
+            assert.isFalse(isFunction(null));
+        });
+    });
+
     describe("#isSet", () => {
         it("should return true for sets", () => {
             assert.isTrue(isSet(new Set()));