feat: loosen up peer api check to require dev ver to satisfy peer ver (…

…#936)

* feat: loosen up peer api check to require dev ver to satisfy peer ver
* fix: install semver locally in scripts

.github/workflows/peer-api.yaml

@@ -18,5 +18,8 @@ jobs:
       - name: Install lerna
         run: npm install -g lerna
 
+      - name: Install script dependencies
+        run: npm install
+
       - name: Check API dependency semantics
         run: lerna exec "node \$LERNA_ROOT_PATH/scripts/peer-api-check.js"

scripts/package.json

@@ -0,0 +1,8 @@
+{
+  "name": "opentelemetry-contrib-scripts",
+  "private": true,
+  "license": "Apache-2.0",
+  "dependencies": {
+    "semver": "7.3.5"
+  }
+}

scripts/peer-api-check.js

@@ -15,6 +15,8 @@
  */
 
 const path = require('path');
+const semver = require('semver');
+
 const appRoot = process.cwd();
 const packageJsonUrl = path.resolve(`${appRoot}/package.json`);
 const pjson = require(packageJsonUrl);
@@ -32,10 +34,10 @@ if (peerVersion) {
       `Package ${pjson.name} does't have API version pinned in dev dependencies: ${devVersion}`
     );
   }
-  if (peerVersion !== `^${devVersion}`) {
+  if (!semver.satisfies(devVersion, peerVersion)) {
     throw new Error(
       `Package ${pjson.name} depends on peer API version ${peerVersion} ` +
-      `but version ${devVersion} in development`
+      `but version ${devVersion} in development which doesn't satisfy the peer API version`
     );
   }
   console.log(`${pjson.name} OK`);