Merge pull request #434 from agiledigital/fix-custom-role-arn

fix: support custom role ARNs

Author: Sean Dawson

.github/dependabot.yml

@@ -1,11 +1,11 @@
 version: 2
 updates:
-- package-ecosystem: npm
-  directory: "/"
-  schedule:
-    interval: daily
-    time: "19:00"
-  open-pull-requests-limit: 10
-  labels:
-    - "dependencies"
-    - "automerge" # Used by Kodiak to work out what to automerge
+  - package-ecosystem: npm
+    directory: "/"
+    schedule:
+      interval: daily
+      time: "19:00"
+    open-pull-requests-limit: 10
+    labels:
+      - "dependencies"
+      - "automerge" # Used by Kodiak to work out what to automerge

.github/workflows/nodejs.yml

@@ -17,7 +17,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [10.x, 12.x, 14.x]
+        node-version: [12.x, 14.x, 16.x]
 
     steps:
       - uses: actions/checkout@v2
@@ -34,11 +34,13 @@ jobs:
       - name: Install dependencies
         run: yarn install --frozen-lockfile
       - name: Build
-        run: yarn run build
+        run: yarn build
       - name: Type Coverage
-        run: yarn run type-coverage
+        run: yarn type-coverage
       - name: Test
-        run: yarn run test
+        run: yarn test --coverage
+      - name: Check formatting
+        run: yarn format-check
       - name: Package example service to ensure compatibility
         run: cd example-service && yarn install --frozen-lockfile && yarn sls package
       - name: Run Semantic Release
@@ -55,4 +57,4 @@ jobs:
           NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
           NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
         run: yarn semantic-release
-        if: matrix.node-version == '12.x'
+        if: matrix.node-version == '14.x'

.gitignore

@@ -20,3 +20,6 @@ yarn-error.log
 
 # workflow specific files
 .envrc
+
+# Jest output
+coverage

.prettierignore

@@ -0,0 +1 @@
+module.exports = { extends: ["@commitlint/config-conventional"] };

commitlint.config.js

@@ -1,4 +1,12 @@
 module.exports = {
-  preset: 'ts-jest',
-  testEnvironment: 'node',
-};
+  preset: "ts-jest",
+  testEnvironment: "node",
+  coverageThreshold: {
+    global: {
+      branches: 87,
+      functions: 66,
+      lines: 78,
+      statements: 76
+    }
+  }
+};