chore: CI check that licenses of all dependencies are at least permissive (#555)

nickevansuk · web-flow · commit 729855cd7585 · 2024-08-06T18:51:08.000+01:00
diff --git a/.github/workflows/code-tests.yml b/.github/workflows/code-tests.yml
@@ -20,5 +20,9 @@ jobs:
         node-version: 18.17.1
     - name: Install OpenActive Test Suite
       run: npm install
+    - name: Check licenses of all dependencies are at least permissive
+      run: |
+        npx npm-reflect install --test
+        for dir in ./packages/*; do (cd "$dir" && npx npm-reflect install --test); done
     - name: Run Checks on the Code (Test the Tests!)
       run: npm test
diff --git a/package.json b/package.json
@@ -4,6 +4,7 @@
   "description": "OpenActive Test Suite",
   "private": true,
   "main": "start.js",
+  "license": "MIT",
   "scripts": {
     "start": "node start.js",
     "debug": "node --inspect start.js -- --runInBand",
@@ -56,5 +57,12 @@
     "hooks": {
       "pre-commit": "npm run gen-and-git-add-everything"
     }
+  },
+  "config": {
+    "allowedLicenseTypes": [
+      "permissive",
+      "publicDomain",
+      "uncategorized"
+    ]
   }
 }
diff --git a/packages/openactive-broker-microservice/package.json b/packages/openactive-broker-microservice/package.json
@@ -59,5 +59,12 @@
   },
   "engines": {
     "node": "18.17.1"
+  },
+  "config": {
+    "allowedLicenseTypes": [
+      "permissive",
+      "publicDomain",
+      "uncategorized"
+    ]
   }
 }
diff --git a/packages/openactive-integration-tests/package.json b/packages/openactive-integration-tests/package.json
@@ -75,5 +75,12 @@
   },
   "engines": {
     "node": "18.17.1"
+  },
+  "config": {
+    "allowedLicenseTypes": [
+      "permissive",
+      "publicDomain",
+      "uncategorized"
+    ]
   }
 }
diff --git a/packages/openactive-openid-browser-automation/package.json b/packages/openactive-openid-browser-automation/package.json
@@ -32,5 +32,12 @@
   },
   "engines": {
     "node": "18.17.1"
+  },
+  "config": {
+    "allowedLicenseTypes": [
+      "permissive",
+      "publicDomain",
+      "uncategorized"
+    ]
   }
 }
diff --git a/packages/openactive-openid-client/package.json b/packages/openactive-openid-client/package.json
@@ -30,5 +30,12 @@
   },
   "engines": {
     "node": "18.17.1"
+  },
+  "config": {
+    "allowedLicenseTypes": [
+      "permissive",
+      "publicDomain",
+      "uncategorized"
+    ]
   }
 }
diff --git a/packages/openactive-openid-test-cli/package.json b/packages/openactive-openid-test-cli/package.json
@@ -33,5 +33,12 @@
   },
   "engines": {
     "node": "18.17.1"
+  },
+  "config": {
+    "allowedLicenseTypes": [
+      "permissive",
+      "publicDomain",
+      "uncategorized"
+    ]
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -59,5 +59,12 @@`
`59`	`59`	`},`
`60`	`60`	`"engines": {`
`61`	`61`	`"node": "18.17.1"`
	`62`	`+ },`
	`63`	`+ "config": {`
	`64`	`+ "allowedLicenseTypes": [`
	`65`	`+ "permissive",`
	`66`	`+ "publicDomain",`
	`67`	`+ "uncategorized"`
	`68`	`+ ]`
`62`	`69`	`}`
`63`	`70`	`}`
Original file line number	Diff line number	Diff line change
`@@ -75,5 +75,12 @@`
`75`	`75`	`},`
`76`	`76`	`"engines": {`
`77`	`77`	`"node": "18.17.1"`
	`78`	`+ },`
	`79`	`+ "config": {`
	`80`	`+ "allowedLicenseTypes": [`
	`81`	`+ "permissive",`
	`82`	`+ "publicDomain",`
	`83`	`+ "uncategorized"`
	`84`	`+ ]`
`78`	`85`	`}`
`79`	`86`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,5 +32,12 @@`
`32`	`32`	`},`
`33`	`33`	`"engines": {`
`34`	`34`	`"node": "18.17.1"`
	`35`	`+ },`
	`36`	`+ "config": {`
	`37`	`+ "allowedLicenseTypes": [`
	`38`	`+ "permissive",`
	`39`	`+ "publicDomain",`
	`40`	`+ "uncategorized"`
	`41`	`+ ]`
`35`	`42`	`}`
`36`	`43`	`}`
Original file line number	Diff line number	Diff line change
`@@ -30,5 +30,12 @@`
`30`	`30`	`},`
`31`	`31`	`"engines": {`
`32`	`32`	`"node": "18.17.1"`
	`33`	`+ },`
	`34`	`+ "config": {`
	`35`	`+ "allowedLicenseTypes": [`
	`36`	`+ "permissive",`
	`37`	`+ "publicDomain",`
	`38`	`+ "uncategorized"`
	`39`	`+ ]`
`33`	`40`	`}`
`34`	`41`	`}`
Original file line number	Diff line number	Diff line change
`@@ -33,5 +33,12 @@`
`33`	`33`	`},`
`34`	`34`	`"engines": {`
`35`	`35`	`"node": "18.17.1"`
	`36`	`+ },`
	`37`	`+ "config": {`
	`38`	`+ "allowedLicenseTypes": [`
	`39`	`+ "permissive",`
	`40`	`+ "publicDomain",`
	`41`	`+ "uncategorized"`
	`42`	`+ ]`
`36`	`43`	`}`
`37`	`44`	`}`