chore: Sync Files - 2022/09/29 -- [bot] Synced file(s) with sammcj/template-repo (#130)

Author: sammcj

.eslintrc.js

@@ -21,7 +21,7 @@ module.exports = {
     sourceType: 'module',
   },
   plugins: ['import', 'prettier'],
-  extends: ['prettier', 'eslint:recommended'],
+  extends: ['plugin:prettier/recommended', 'eslint:recommended'],
   root: true,
   rules: {
     quotes: ['error', 'single', { avoidEscape: true }],
@@ -32,10 +32,7 @@ module.exports = {
     'object-curly-spacing': ['error', 'always'],
     'array-bracket-spacing': ['error', 'never'],
     'object-curly-newline': ['error', { multiline: true, consistent: true }],
-    'object-property-newline': [
-      'error',
-      { allowAllPropertiesOnSameLine: true },
-    ],
+    'object-property-newline': ['error', { allowAllPropertiesOnSameLine: true }],
     'keyword-spacing': ['error'],
     'brace-style': ['error', '1tbs', { allowSingleLine: true }],
     'space-before-blocks': 'error',
@@ -46,15 +43,12 @@ module.exports = {
     'no-shadow': 'off',
     'no-use-before-define': 'off',
     'import/order': 'error',
+    'prettier/prettier': 'error',
     'max-classes-per-file': ['error', 3],
     /** no console and debugger in CDK  */
     'no-console': 'error',
     'no-debugger': 'error',
     'no-underscore-dangle': 'off',
-    'lines-between-class-members': [
-      'error',
-      'always',
-      { exceptAfterSingleLine: true },
-    ],
+    'lines-between-class-members': ['error', 'always', { exceptAfterSingleLine: true }],
   },
 };

.github/config/labeller.yml

@@ -44,15 +44,7 @@ labels:
       baseBranch: "(^docs\/.*)/i"
       commits: "(^docs:.*|(IF-[0-9]+ -- docs.*))/i"
       files:
-        any:
-          [
-            "**/docs/**",
-            "**/doc/**",
-            "**/*.md",
-            "**/*.markdown",
-            "**/*.rst",
-            "**/*.txt",
-          ]
+        any: ["**/docs/**", "**/doc/**", "**/*.md", "**/*.markdown", "**/*.rst", "**/*.txt"]
         count:
           gte: 1
           lte: 1000
@@ -140,15 +132,7 @@ labels:
       commits: "(^db:.*|.*sql|(IF-[0-9]+ -- db.*))/i"
       files:
         # regex glob to match a file with extension .sql in any directory recursively
-        any:
-          [
-            "**/*.sql",
-            "**/*.db",
-            "**/db/**",
-            "**/flyway.*",
-            "**/migrations/**",
-            "**/*.sqlite.*",
-          ]
+        any: ["**/*.sql", "**/*.db", "**/db/**", "**/flyway.*", "**/migrations/**", "**/*.sqlite.*"]
         count:
           gte: 1
           lte: 1000
@@ -183,7 +167,7 @@ labels:
     matcher:
       title: ".*git.*/i"
       files:
-        any: ["**/.git.*", "**/.husky/*"]
+        any: ["**/.git.*", "**/.husky/*", "**/.pre-commit-config.*"]
         count:
           gte: 1
           lte: 1000

.github/workflows/lint-tf.yml

@@ -22,7 +22,9 @@ jobs:
       # Note: Terraform is not needed for tflint
       - name: Install Terraform
         run: |
-          brew install terraform
+          curl -sSL https://apt.releases.hashicorp.com/gpg | sudo apt-key add -
+          sudo apt-add-repository "deb [arch=amd64] https://apt.releases.hashicorp.com $(lsb_release -cs) main"
+          sudo apt-get update && sudo apt-get install terraform -y
       # Run init to get module code to be able to use `--module`
       - name: Terraform init
         run: |

.pre-commit-config.yaml

@@ -0,0 +1,36 @@
+# See https://pre-commit.com for more information
+# See https://pre-commit.com/hooks.html for more hooks
+repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.3.0
+    hooks:
+      - id: trailing-whitespace
+      - id: end-of-file-fixer
+      - id: check-yaml
+      - id: check-added-large-files
+      - id: check-case-conflict
+      - id: check-json
+      - id: check-merge-conflict
+      - id: check-symlinks
+      - id: check-toml
+      - id: check-xml
+      - id: detect-private-key
+      - id: check-added-large-files
+
+  - repo: https://github.com/pre-commit/mirrors-eslint
+    rev: v8.24.0
+    hooks:
+      - id: eslint
+        files: \.[jt]sx?$ # *.js, *.jsx, *.ts and *.tsx
+        types: [file]
+        additional_dependencies:
+          - [email protected]
+          - [email protected]
+          - [email protected]
+          - [email protected]
+
+  - repo: https://github.com/syntaqx/git-hooks
+    rev: v0.0.17
+    hooks:
+      - id: shellcheck
+      - id: shfmt

commitlint.config.js

@@ -0,0 +1,128 @@
+module.exports = {
+  parserPreset: 'conventional-changelog-conventionalcommits',
+  rules: {
+    'body-leading-blank': [1, 'always'],
+    'body-max-line-length': [2, 'always', 100],
+    'footer-leading-blank': [1, 'always'],
+    'footer-max-line-length': [2, 'always', 100],
+    'header-max-length': [2, 'always', 100],
+    'subject-case': [2, 'never', ['sentence-case', 'start-case', 'pascal-case', 'upper-case']],
+    'subject-empty': [2, 'never'],
+    'subject-full-stop': [2, 'never', '.'],
+    'type-case': [2, 'always', 'lower-case'],
+    'type-empty': [2, 'never'],
+    'type-enum': [
+      2,
+      'always',
+      [
+        'build',
+        'chore',
+        'ci',
+        'docs',
+        'feat',
+        'fix',
+        'perf',
+        'refactor',
+        'revert',
+        'style',
+        'test',
+      ],
+    ],
+  },
+  prompt: {
+    questions: {
+      type: {
+        description: "Select the type of change that you're committing",
+        enum: {
+          feat: {
+            description: 'A new feature',
+            title: 'Features',
+            emoji: '✨',
+          },
+          fix: {
+            description: 'A bug fix',
+            title: 'Bug Fixes',
+            emoji: '🐛',
+          },
+          docs: {
+            description: 'Documentation only changes',
+            title: 'Documentation',
+            emoji: '📚',
+          },
+          style: {
+            description:
+              'Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)',
+            title: 'Styles',
+            emoji: '💎',
+          },
+          refactor: {
+            description: 'A code change that neither fixes a bug nor adds a feature',
+            title: 'Code Refactoring',
+            emoji: '📦',
+          },
+          perf: {
+            description: 'A code change that improves performance',
+            title: 'Performance Improvements',
+            emoji: '🚀',
+          },
+          test: {
+            description: 'Adding missing tests or correcting existing tests',
+            title: 'Tests',
+            emoji: '🚨',
+          },
+          build: {
+            description:
+              'Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)',
+            title: 'Builds',
+            emoji: '🛠',
+          },
+          ci: {
+            description:
+              'Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)',
+            title: 'Continuous Integrations',
+            emoji: '⚙️',
+          },
+          chore: {
+            description: "Other changes that don't modify src or test files",
+            title: 'Chores',
+            emoji: '♻️',
+          },
+          revert: {
+            description: 'Reverts a previous commit',
+            title: 'Reverts',
+            emoji: '🗑',
+          },
+        },
+      },
+      scope: {
+        description: 'What is the scope of this change (e.g. component or file name)',
+      },
+      subject: {
+        description: 'Write a short, imperative tense description of the change',
+      },
+      body: {
+        description: 'Provide a longer description of the change',
+      },
+      isBreaking: {
+        description: 'Are there any breaking changes?',
+      },
+      breakingBody: {
+        description:
+          'A BREAKING CHANGE commit requires a body. Please enter a longer description of the commit itself',
+      },
+      breaking: {
+        description: 'Describe the breaking changes',
+      },
+      isIssueAffected: {
+        description: 'Does this change affect any open issues?',
+      },
+      issuesBody: {
+        description:
+          'If issues are closed, the commit requires a body. Please enter a longer description of the commit itself',
+      },
+      issues: {
+        description: 'Add issue references (e.g. "fix #123", "re #123".)',
+      },
+    },
+  },
+};