Merge pull request #8 from SimonGolms/feature/dependencies

feature/dependencies

Author: SimonGolms

.commitlintrc.js

@@ -1,8 +1,8 @@
 /* eslint-disable import/no-unused-modules */
 module.exports = {
-  extends: ['@commitlint/config-conventional'],
+  extends: ["@commitlint/config-conventional"],
   // See: https://commitlint.js.org/#/reference-rules
   rules: {
-    'body-max-line-length': [0, 'always', Infinity], // Disabled rule, since the release notes via semantic-release in the commit body message can be appropriately long.
+    "body-max-line-length": [0, "always", Infinity], // Disabled rule, since the release notes via semantic-release in the commit body message can be appropriately long.
   },
 };

.eslintrc.js

@@ -90,8 +90,8 @@ module.exports = {
         alphabetize: {
           caseInsensitive: true,
           order: "asc",
+          orderImportKind: "asc",
         },
-        groups: ["builtin", "external", "internal"],
         pathGroups: [
           {
             group: "external",
@@ -104,8 +104,10 @@ module.exports = {
     ],
     "import/prefer-default-export": "off",
     "no-console": ["warn", { allow: ["warn", "error"] }],
+    "playwright/prefer-to-be": "error",
+    "playwright/prefer-to-have-length": "error",
+    "playwright/require-top-level-describe": "error",
     "prettier/prettier": "error",
-    "react/jsx-sort-default-props": "error",
     "react/jsx-sort-props": [
       "error",
       {
@@ -117,6 +119,7 @@ module.exports = {
         shorthandLast: false,
       },
     ],
+    "react/sort-default-props": "error",
     "react-hooks/exhaustive-deps": "error",
     "react-hooks/rules-of-hooks": "error",
     "sort-imports": [

.github/.releaserc.js

@@ -1,73 +1,74 @@
 module.exports = {
-  branches: ['main'],
+  branches: ["main"],
   plugins: [
     [
-      '@semantic-release/commit-analyzer',
+      "@semantic-release/commit-analyzer",
       {
         parserOpts: {
-          noteKeywords: ['BREAKING CHANGE', 'BREAKING CHANGES', 'BREAKING'],
+          noteKeywords: ["BREAKING CHANGE", "BREAKING CHANGES", "BREAKING"],
         },
-        preset: 'angular',
+        preset: "angular",
         releaseRules: [
-          { breaking: true, release: 'major' },
-          { release: 'patch', type: 'chore' },
-          { release: 'patch', type: 'ci' },
-          { release: 'patch', type: 'refactor' },
-          { release: 'patch', type: 'style' },
+          { breaking: true, release: "major" },
+          { release: "patch", type: "chore" },
+          { release: "patch", type: "ci" },
+          { release: "patch", type: "refactor" },
+          { release: "patch", type: "style" },
         ],
       },
     ],
     [
-      '@semantic-release/release-notes-generator',
+      "@semantic-release/release-notes-generator",
       {
         parserOpts: {
-          noteKeywords: ['BREAKING CHANGE', 'BREAKING CHANGES', 'BREAKING'],
+          noteKeywords: ["BREAKING CHANGE", "BREAKING CHANGES", "BREAKING"],
         },
-        preset: 'conventionalcommits',
+        preset: "conventionalcommits",
         presetConfig: {
           types: [
-            { hidden: false, section: 'Build System', type: 'build' },
-            { hidden: false, section: 'Build System', type: 'chore' },
-            { hidden: false, section: 'Continuous Integration', type: 'ci' },
-            { hidden: false, section: 'Documentation', type: 'docs' },
-            { hidden: false, section: 'Features', type: 'feat' },
-            { hidden: false, section: 'Bug Fixes', type: 'fix' },
+            { hidden: false, section: "Build System", type: "build" },
+            { hidden: false, section: "Build System", type: "chore" },
+            { hidden: false, section: "Continuous Integration", type: "ci" },
+            { hidden: false, section: "Documentation", type: "docs" },
+            { hidden: false, section: "Features", type: "feat" },
+            { hidden: false, section: "Bug Fixes", type: "fix" },
             {
               hidden: false,
-              section: 'Performance Improvements',
-              type: 'perf',
+              section: "Performance Improvements",
+              type: "perf",
             },
-            { hidden: false, section: 'Code Refactoring', type: 'refactor' },
-            { hidden: false, section: 'Styles', type: 'style' },
-            { hidden: false, section: 'Tests', type: 'test' },
+            { hidden: false, section: "Code Refactoring", type: "refactor" },
+            { hidden: false, section: "Styles", type: "style" },
+            { hidden: false, section: "Tests", type: "test" },
           ],
         },
         writerOpts: {
-          commitsSort: ['subject', 'scope'],
+          commitsSort: ["subject", "scope"],
         },
       },
     ],
     [
-      '@semantic-release/exec',
+      "@semantic-release/exec",
       {
         // HINT: Make sure that the changed files are part of the 'assets' property in @semantic-release/git
-        verifyReleaseCmd: "sed -i 's/version-.*-blue/version-${nextRelease.version.replace(/-/g, '--')}-blue/g' README.md",
+        verifyReleaseCmd:
+          "sed -i 's/version-.*-blue/version-${nextRelease.version.replace(/-/g, '--')}-blue/g' README.md",
       },
     ],
     [
-      '@semantic-release/changelog',
+      "@semantic-release/changelog",
       {
-        changelogFile: 'CHANGELOG.md',
-        changelogTitle: '# Changelog',
+        changelogFile: "CHANGELOG.md",
+        changelogTitle: "# Changelog",
       },
     ],
     [
-      '@semantic-release/git',
+      "@semantic-release/git",
       {
-        assets: ['CHANGELOG.md', 'README.md'],
-        message: 'chore(release): ${nextRelease.version} [skip ci]\n\n${nextRelease.notes}',
+        assets: ["CHANGELOG.md", "README.md"],
+        message: "chore(release): ${nextRelease.version} [skip ci]\n\n${nextRelease.notes}",
       },
     ],
-    '@semantic-release/github',
+    "@semantic-release/github",
   ],
 };

.github/ISSUE_TEMPLATE/bug-report.yaml

@@ -1,6 +1,6 @@
 name: 🐛 Bug Report
 description: Create a report to help to improve the ODS Quickstarter
-title: 'bug: '
+title: "bug: "
 body:
   - type: checkboxes
     attributes:

.github/ISSUE_TEMPLATE/feature-request.yaml

@@ -1,6 +1,6 @@
 name: 💡 Feature Request
 description: Suggest an idea for the ODS Quickstarter
-title: 'feat: '
+title: "feat: "
 body:
   - type: checkboxes
     attributes:

.github/workflows/build-and-test.yaml

@@ -2,16 +2,16 @@ name: Build and Test
 on:
   push:
     branches-ignore:
-      - 'main'
+      - "main"
 
 jobs:
   build_and_test:
-    name: 'Build and Test'
+    name: "Build and Test"
     runs-on: ubuntu-20.04
     container:
       # Using the playwright container (https://mcr.microsoft.com/product/playwright/about)
       # for performance reasons to avoid having to download and install all dependencies again.
-      image: mcr.microsoft.com/playwright:v1.27.1-focal
+      image: mcr.microsoft.com/playwright:v1.29.2-focal
       # Container should run with the same user of the host VM, otherwise it leads to permission issues
       # see: https://github.com/actions/runner/issues/691 & https://github.com/actions/checkout/issues/211#issuecomment-680107607
       options: --user 1001
@@ -21,9 +21,9 @@ jobs:
       - name: Setup Node.js
         uses: actions/setup-node@v3
         with:
-          node-version-file: '.nvmrc'
-          cache: 'npm'
-          cache-dependency-path: '**/package-lock.json'
+          node-version-file: ".nvmrc"
+          cache: "npm"
+          cache-dependency-path: "**/package-lock.json"
       # Ignore README.md from find results to avoid unintentionally changes on the release step
       - name: Setup Quickstarter (Project ID)
         run: find . -type f \( ! -name "README.md" \) -exec sed -i 's/PROJECTID/foo/g' {} +
@@ -36,9 +36,9 @@ jobs:
         run: npm run lint
       - name: Test Components
         env:
-          VITE_AZURE_ACTIVE_DIRECTORY_CLIENT_ID: '11111111-2222-3333-4444-555555555dev'
+          VITE_AZURE_ACTIVE_DIRECTORY_CLIENT_ID: "11111111-2222-3333-4444-555555555dev"
           # IMPORTANT: A valid Azure AD Tenant ID for testing purposes is required.
-          VITE_AZURE_ACTIVE_DIRECTORY_TENANT_ID: 'common'
+          VITE_AZURE_ACTIVE_DIRECTORY_TENANT_ID: "common"
         run: npm run test
       - name: Build Application
         run: npm run build

.github/workflows/release.yaml

@@ -16,9 +16,9 @@ jobs:
       - name: Setup Node.js
         uses: actions/setup-node@v3
         with:
-          node-version-file: '.nvmrc'
-          cache: 'npm'
-          cache-dependency-path: '**/package-lock.json'
+          node-version-file: ".nvmrc"
+          cache: "npm"
+          cache-dependency-path: "**/package-lock.json"
       # Ignore README.md from find results to avoid unintentionally changes on the release step
       - name: Setup Quickstarter (Project ID)
         run: find . -type f \( ! -name "README.md" \) -exec sed -i 's/PROJECTID/foo/g' {} +
@@ -29,9 +29,9 @@ jobs:
         run: npm ci
       - name: Release
         env:
-          GIT_AUTHOR_NAME: 'Simon Golms'
-          GIT_AUTHOR_EMAIL: 'development@gol.ms'
-          GIT_COMMITTER_NAME: 'Simon Golms'
-          GIT_COMMITTER_EMAIL: 'development@gol.ms'
+          GIT_AUTHOR_NAME: "Simon Golms"
+          GIT_AUTHOR_EMAIL: "development@gol.ms"
+          GIT_COMMITTER_NAME: "Simon Golms"
+          GIT_COMMITTER_EMAIL: "development@gol.ms"
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: npx semantic-release --extends './.github/.releaserc.js'

.lintstagedrc.js

@@ -1,14 +1,14 @@
 module.exports = {
-  '*.css': ['stylelint --fix', 'git add --force'],
-  '*.{js,jsx}': [
+  "*.css": ["stylelint --fix", "git add --force"],
+  "*.{js,jsx}": [
     // Extend rule set of .eslintrc.js with 'no-console' and 'no-unused-vars'
     "eslint --fix --rule 'no-console: [error, { allow: [warn, error] }]' --rule 'no-unused-vars: error'",
-    'git add --force',
+    "git add --force",
   ],
-  '*.{ts,tsx}': [
+  "*.{ts,tsx}": [
     // Extend rule set of .eslintrc.js with 'no-console' and '@typescript-eslint/no-unused-vars'
     "eslint --fix --rule 'no-console: [error, { allow: [warn, error] }]' --rule '@typescript-eslint/no-unused-vars: [error, { argsIgnorePattern: ^_, caughtErrorsIgnorePattern: ^_, varsIgnorePattern: ^_, }]'",
-    'git add --force',
+    "git add --force",
   ],
-  '*.json': ['prettier --write', 'git add --force'],
+  "*.json": ["prettier --write", "git add --force"],
 };

.prettierrc.js

@@ -1 +1 @@
-module.exports = require('@boehringer-ingelheim/prettier-config');
+module.exports = require("@boehringer-ingelheim/prettier-config");

.stylelintrc.js

@@ -1,15 +1,15 @@
 module.exports = {
-  extends: ['stylelint-config-standard', 'stylelint-prettier/recommended'],
-  plugins: ['stylelint-order', 'stylelint-prettier'],
+  extends: ["stylelint-config-standard", "stylelint-prettier/recommended"],
+  plugins: ["stylelint-order", "stylelint-prettier"],
   rules: {
     /**
      * Ionic uses colors with an opacity (alpha) in several components.
      * In order for this to work, those properties must be provided in RGB format
      * in a comma separated format without parentheses.
      * see: https://ionicframework.com/docs/theming/advanced#the-alpha-problem
      */
-    'color-function-notation': 'legacy',
-    'order/order': ['custom-properties', 'declarations'],
-    'order/properties-alphabetical-order': true,
+    "color-function-notation": "legacy",
+    "order/order": ["custom-properties", "declarations"],
+    "order/properties-alphabetical-order": true,
   },
 };