feat!: throw for versions of Node.js older than v10.0.0 (#748)

Benjamin E. Coe · alexander-fenster · web-flow · commit 511fc233bd66 · 2020-03-19T18:04:53.000-07:00
* feat!: throw for versions of Node.js older than v10.0.0

* chore: address code review

* chore: add a space

* chore: remove node 8 from CI

Co-authored-by: Alexander Fenster &lt;fenster@google.com&gt;
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -9,7 +9,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        node: [8, 10, 12, 13]
+        node: [10, 12, 13]
     steps:
       - uses: actions/checkout@v1
       - uses: actions/setup-node@v1
diff --git a/src/grpc.ts b/src/grpc.ts
@@ -97,19 +97,20 @@ export class GrpcClient {
     this.auth = options.auth || new GoogleAuth(options);
     this.fallback = false;
 
+    const minimumVersion = '10.0.0';
+    if (semver.lt(process.version, minimumVersion)) {
+      const errorMessage =
+        `Node.js v${minimumVersion} is a minimum requirement. To learn about legacy version support visit: ` +
+        'https://github.com/googleapis/google-cloud-node#supported-nodejs-versions';
+      throw new Error(errorMessage);
+    }
+
     if ('grpc' in options) {
       this.grpc = options.grpc!;
       this.grpcVersion = '';
     } else {
-      if (semver.gte(process.version, '8.13.0')) {
-        this.grpc = grpc;
-        this.grpcVersion = require('@grpc/grpc-js/package.json').version;
-      } else {
-        const errorMessage =
-          'To use @grpc/grpc-js you must run your code on Node.js v8.13.0 or newer. Please see README if you need to use an older version. ' +
-          'https://github.com/googleapis/gax-nodejs/blob/master/README.md';
-        throw new Error(errorMessage);
-      }
+      this.grpc = grpc;
+      this.grpcVersion = require('@grpc/grpc-js/package.json').version;
     }
   }
 
