🤖 Release PR

.all-contributorsrc

@@ -0,0 +1,65 @@
+{
+  "files": [
+    "README.md"
+  ],
+  "imageSize": 60,
+  "contributorsPerLine": 7,
+  "contributorsSortAlphabetically": false,
+  "badgeTemplate": "[![All Contributors](https://img.shields.io/badge/all_contributors-<%= contributors.length %>-orange.svg?style=flat-square)](#contributors)",
+  "skipCi": true,
+  "types": {
+    "maintainer": {
+      "symbol": "💪",
+      "description": "Maintainer",
+      "link": "[<%= symbol %>](\"<%= description %>\"),"
+    }
+  },
+  "contributors": [
+    {
+      "login": "davehakkens",
+      "name": "Dave Hakkens",
+      "avatar_url": "https://avatars.githubusercontent.com/u/13672737?v=4",
+      "profile": "http://davehakkens.nl",
+      "contributions": [
+        "design",
+        "ideas",
+        "projectManagement",
+        "maintainer"
+      ]
+    },
+    {
+      "login": "chrismclarke",
+      "name": "Chris Clarke",
+      "avatar_url": "https://avatars.githubusercontent.com/u/10515065?v=4",
+      "profile": "https://c2dev.co.uk/",
+      "contributions": [
+        "code",
+        "maintainer"
+      ]
+    },
+    {
+      "login": "thisislawatts",
+      "name": "Luke Watts",
+      "avatar_url": "https://avatars.githubusercontent.com/u/472589?v=4",
+      "profile": "https://thisis.la/",
+      "contributions": [
+        "code",
+        "maintainer"
+      ]
+    },
+    {
+      "login": "amuroBosetti",
+      "name": "Mauro Bosetti",
+      "avatar_url": "https://avatars.githubusercontent.com/u/46928545?v=4",
+      "profile": "https://github.com/amuroBosetti",
+      "contributions": [
+        "doc"
+      ]
+    }
+  ],
+  "projectName": "community-platform",
+  "projectOwner": "ONEARMY",
+  "repoType": "github",
+  "repoHost": "https://github.com",
+  "commitConvention": "none"
+}

.circleci/config.yml

@@ -195,6 +195,17 @@ examples:
 #  Jobs - Independently specified lists of tasks and environments for execution
 ######################################################################################################
 jobs:
+  spellcheck:
+    docker: *docker
+    resource_class: small
+    steps:
+      - checkout
+      - attach_workspace:
+          at: '.'
+      - run:
+          name: Check spelling on documentation
+          command: |
+            npx cspell "**/*.md" --config ./.cspell.json
   lint-commits:
     docker: *docker
     resource_class: small
@@ -209,6 +220,12 @@ jobs:
             npm config set package-lock false
             npm i @types/node @commitlint/types @commitlint/config-conventional --force
             npx commitlint --from=$(git merge-base remotes/origin/${CIRCLE_BRANCH} master) --verbose
+  lint:
+    docker: *docker
+    steps:
+      - setup_repo
+      - run:
+          command: yarn lint
   # Prepare node module caches so that future tasks run more quickly
   # NOTE - not currently used as we only have one workflow
   setup:
@@ -365,40 +382,53 @@ workflows:
   main_workflow:
     # by default jobs will run concurrently, so specify requires if want to run sequentially
     jobs:
+      - spellcheck:
+          name: Check documentation spelling
+          filters:
+            branches:
+              ignore:
+                - production
       - lint-commits:
           name: Lint commits
           filters:
             branches:
               ignore:
                 - production
+      - lint:
+          name: Lint code
+          requires:
+            - 'Check documentation spelling'
+            - 'Lint commits'
+          filters:
+            branches:
+              ignore:
+                - production
       #---------------------- Test ----------------------
       # Note - when calling test we also let the test script handle building as it injects random variables for seeding the DB
       - build:
           requires:
-            - 'Lint commits'
+            - 'Lint code'
           name: Build Application
           context: build-context
       - test_unit:
           name: 'Unit tests'
           requires:
-            - 'Lint commits'
+            - 'Lint code'
           filters:
             branches:
               ignore:
                 - production
       - storybook:
           name: Build Storybook
           requires:
-            - 'Lint commits'
+            - 'Lint code'
           filters:
             branches:
               ignore:
                 - production
       # - test_functions:
       #     name: 'Cloud Function tests'
       #     context: build-context
-      #     requires:
-      #       - 'Lint commits'
       #     filters:
       #       branches:
       #         ignore:
@@ -421,10 +451,17 @@ workflows:
               ignore:
                 - production
       #---------------------- Development Instances Build and Deploy ----------------------
+      - release:
+          name: Release new version to GitHub
+          requires:
+            - test_e2e
+          <<: *filter_only_master
+          context:
+            - release-context
       - deploy:
           name: 'Deploy: dev.onearmy.world'
           requires:
-            - test_e2e
+            - 'Release new version to GitHub'
           <<: *filter_only_master
           DEPLOY_ALIAS: 'default'
           NOTIFY_SLACK: false
@@ -434,24 +471,37 @@ workflows:
       - deploy:
           name: 'Deploy: dev.community.projectkamp.com'
           requires:
-            - test_e2e
+            - 'Release new version to GitHub'
           <<: *filter_only_master
           DEPLOY_ALIAS: project-kamp-development
           NOTIFY_SLACK: false
           context:
             - circle-ci-slack-context
             - project-kamp-dev
-      - release:
+      - deploy:
+          name: 'Deploy: fixing-fashion-dev.web.app'
           requires:
-            - 'Deploy: community.preciousplastic.com'
-            - 'Deploy: community.projectkamp.com'
-          name: Release new version to GitHub
+            - 'Release new version to GitHub'
+          <<: *filter_only_master
+          DEPLOY_ALIAS: fixing-fashion-dev
+          NOTIFY_SLACK: false
           context:
-            - release-context
+            - circle-ci-slack-context
+            - fixing-fashion-dev
       - build:
           name: Build Production Release
           context: build-context
           <<: *filter_only_production
+      - deploy:
+          name: 'Deploy: fixing-fashion-prod.web.app'
+          requires:
+            - 'Build Production Release'
+          <<: *filter_only_production
+          DEPLOY_ALIAS: fixing-fashion-prod
+          NOTIFY_SLACK: false
+          context:
+            - circle-ci-slack-context
+            - fixing-fashion-prod
       - deploy:
           name: 'Deploy: community.preciousplastic.com'
           requires:

.cspell.json

@@ -0,0 +1,43 @@
+{
+  "version": "0.2",
+  "language": "en-GB",
+  "ignorePaths": ["node_modules/**", "**/node_modules/**"],
+  "words": [
+    "appengine",
+    "appspot",
+    "bigquery",
+    "blockquotes",
+    "boxen",
+    "circleci",
+    "davehakkens",
+    "dockerhub",
+    "dockerised",
+    "dockerode",
+    "enoent",
+    "firebase",
+    "firestore",
+    "franknoirot",
+    "formattedly",
+    "gcloud",
+    "globby",
+    "gsutil",
+    "hakkens",
+    "howto",
+    "howtos",
+    "indexeddb",
+    "kamp",
+    "mappins",
+    "mattia",
+    "microservices",
+    "mockup",
+    "mockups",
+    "onearmyworld",
+    "phonebloks",
+    "preciousplastic",
+    "projectkamp",
+    "redownload",
+    "runtimeconfig",
+    "strikethrough",
+    "taskkill"
+  ]
+}

.eslintrc.json

@@ -1,6 +1,6 @@
 {
   "parser": "@typescript-eslint/parser",
-  "plugins": ["@typescript-eslint", "prettier"],
+  "plugins": ["@typescript-eslint", "prettier", "unicorn"],
   "extends": [
     "plugin:react/recommended", // Uses the recommended rules from @eslint-plugin-react
     "plugin:@typescript-eslint/recommended", // Uses the recommended rules from the @typescript-eslint/eslint-plugin
@@ -23,6 +23,12 @@
     // as of v17 no longer required
     "react/react-in-jsx-scope": "off",
     "react/prop-types": "off",
+    "@typescript-eslint/consistent-type-imports": [
+      "error",
+      {
+        "prefer": "type-imports"
+      }
+    ],
     "@typescript-eslint/explicit-function-return-type": "off",
     "@typescript-eslint/explicit-module-boundary-types": "off",
     "@typescript-eslint/interface-name-prefix": "off",
@@ -41,6 +47,39 @@
         "leadingUnderscore": "allow"
       }
     ],
+    // Filenames - https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/filename-case.md
+    "unicorn/filename-case": [
+      "error",
+      {
+        "cases": {
+          "camelCase": true,
+          "pascalCase": true
+        },
+        // TODO - most can be renamed although holding off until after #1501 and #1533 merged
+        "ignore": [
+          "docusaurus.config.gh-pages.js",
+          "install-clean.ts",
+          "SWUpdateNotification.tsx",
+          "react-app-env.d.ts",
+          "service-worker.ts",
+          "AbstractDBClient.ts",
+          "react-firebase-file-uploader.d.ts",
+          "reset-staging-site.ts",
+          "post-cra-build.ts",
+          "user_pp.mock.tsx",
+          "user_pp.models.tsx",
+          "admin-subheader.tsx",
+          "admin-approvals.tsx",
+          "admin-howtos.tsx",
+          "admin-mappins.tsx",
+          "admin-research-detail.tsx",
+          "admin-research.tsx",
+          "admin-tags.tsx",
+          "admin-user-detail.tsx",
+          "admin-users.tsx"
+        ]
+      }
+    ],
     "@typescript-eslint/no-empty-function": "off",
     "@typescript-eslint/no-explicit-any": "off",
     "@typescript-eslint/no-inferrable-types": "off",

.github/ISSUE_TEMPLATE/bug_report.md

@@ -10,14 +10,14 @@ assignees: ''
 A clear and concise description of what the bug is.
 
 **To Reproduce**
-Steps to reproduce the behavior:
+Steps to reproduce the behaviour:
 
 1. Go to '...'
 2. Click on '....'
 3. Scroll down to '....'
 4. See error
 
-**Expected behavior**
+**Expected behaviour**
 A clear and concise description of what you expected to happen.
 
 **Screenshots**