chore: migrate from cli-style to the shared style configs

.github/pull_request_template.md

@@ -10,15 +10,15 @@ _text_
 
 ### Known issues
 
--   [ ] _issue_
+- [ ] _issue_
 
 ---
 
 ### Checklist
 
--   [ ] API docs are generated
--   [ ] Tests were added
--   [ ] Storybook demos were added
+- [ ] API docs are generated
+- [ ] Tests were added
+- [ ] Storybook demos were added
 
 _All points above should be relevant for feature PRs. For bugfixes, some points might not be relevant. In that case, just check them anyway to signal the work is done._
 

.github/workflows/dhis2-deploy-netlify.yml

@@ -26,7 +26,7 @@ jobs:
             - uses: actions/checkout@v2
             - uses: actions/setup-node@v3
               with:
-                  node-version: 20.x
+                  node-version: 24.x
 
             # We use Netlify only for deploy previews, production docs are hosted at developers.dhis2.org
             # Don't build anything, just copy the _redirects file into the dist directory

.github/workflows/dhis2-preview-pr.yml

@@ -24,7 +24,7 @@ jobs:
             - uses: actions/checkout@v2
             - uses: actions/setup-node@v3
               with:
-                  node-version: 20.x
+                  node-version: 24.x
                   cache: 'yarn'
 
             - run: yarn install --frozen-lockfile

.github/workflows/dhis2-verify-commits.yml

@@ -11,15 +11,17 @@ jobs:
             - uses: actions/checkout@v2
             - uses: actions/setup-node@v3
               with:
-                  node-version: 20.x
+                  node-version: 24.x
                   cache: 'yarn'
 
             - run: yarn install --frozen-lockfile
-            - id: commitlint
-              run: echo ::set-output name=config_path::$(node -e "process.stdout.write(require('@dhis2/cli-style').config.commitlint)")
-            - uses: JulienKode/pull-request-name-linter-action@v0.5.0
-              with:
-                  configuration-path: ${{ steps.commitlint.outputs.config_path }}
+
+            # The title is untrusted input, so it reaches the shell through
+            # the environment.
+            - name: Lint pull request title
+              env:
+                  PR_TITLE: ${{ github.event.pull_request.title }}
+              run: echo "$PR_TITLE" | yarn commitlint
 
     lint-commits:
         runs-on: ubuntu-latest
@@ -29,12 +31,8 @@ jobs:
                   fetch-depth: 0
             - uses: actions/setup-node@v3
               with:
-                  node-version: 20.x
+                  node-version: 24.x
                   cache: 'yarn'
 
             - run: yarn install --frozen-lockfile
-            - id: commitlint
-              run: echo ::set-output name=config_path::$(node -e "process.stdout.write(require('@dhis2/cli-style').config.commitlint)")
-            - uses: wagoid/commitlint-github-action@v5
-              with:
-                  configFile: ${{ steps.commitlint.outputs.config_path }}
+            - uses: wagoid/commitlint-github-action@v6

.github/workflows/dhis2-verify-lib.yml

@@ -36,7 +36,7 @@ jobs:
             - uses: actions/checkout@v2
             - uses: actions/setup-node@v3
               with:
-                  node-version: 20.x
+                  node-version: 24.x
                   cache: 'yarn'
 
             - run: |
@@ -61,7 +61,7 @@ jobs:
             - uses: actions/checkout@v2
             - uses: actions/setup-node@v3
               with:
-                  node-version: 20.x
+                  node-version: 24.x
                   cache: 'yarn'
 
             - uses: actions/download-artifact@v4
@@ -87,7 +87,7 @@ jobs:
             - uses: actions/checkout@v2
             - uses: actions/setup-node@v3
               with:
-                  node-version: 20.x
+                  node-version: 24.x
                   cache: 'yarn'
 
             - uses: actions/download-artifact@v4
@@ -118,7 +118,7 @@ jobs:
 
             - uses: actions/setup-node@v3
               with:
-                  node-version: 20.x
+                  node-version: 24.x
 
             - uses: actions/download-artifact@v4
               with:
@@ -178,7 +178,7 @@ jobs:
                   token: ${{ env.GH_TOKEN }}
             - uses: actions/setup-node@v3
               with:
-                  node-version: 20.x
+                  node-version: 24.x
                   cache: 'yarn'
             - uses: dhis2/action-commit-signing@v1
               with:

.github/workflows/generate-and-upload-bom.yml

@@ -20,7 +20,7 @@ jobs:
             - uses: actions/checkout@v4
             - uses: actions/setup-node@v4
               with:
-                  node-version: 20.x
+                  node-version: 24.x
 
             - name: Install Dependencies
               run: yarn install --frozen-lockfile

.hooks/commit-msg

@@ -1,4 +1,3 @@
 #!/bin/sh
-. "$(dirname "$0")/_/husky.sh"
 
-yarn d2-style check commit "$1"
+yarn commitlint --edit "$1"

.hooks/pre-commit

@@ -1,4 +1,20 @@
 #!/bin/sh
-. "$(dirname "$0")/_/husky.sh"
+set -e
 
-yarn d2-style check --staged
+staged_files_without_deletions() {
+    git diff --cached --name-only --diff-filter=d | grep -E "$1" || true
+}
+
+check() {
+    files=$1
+    shift
+
+    if [ -n "$files" ]; then
+        printf '%s\n' "$files" | tr '\n' '\0' | xargs -0 "$@"
+    fi
+}
+
+check "$(staged_files_without_deletions '\.(js|jsx|ts|tsx)$')" \
+    yarn eslint --no-warn-ignored
+check "$(staged_files_without_deletions '\.(js|jsx|ts|tsx|md|json|yml|html|css)$')" \
+    yarn prettier --check --ignore-unknown

.nvmrc

@@ -1 +1 @@
-v20
+v24

.prettierignore

@@ -1,12 +1,20 @@
 .yarn/
+.d2/
 **/locales/
 cypress/assets/*.js
 
 # Documentation files that are generated on CI
 API.md
 
+# Generated from the SVG sources by `yarn setup`
+icons/src/react/
+
 # Build output, generated locally by `yarn build` (docusaurus site into dist/,
 # its build cache into docs/.docusaurus/). Without these, `yarn build` followed
 # by `yarn lint` fails on generated files nobody wrote.
 dist/
+**/build/
 **/.docusaurus/
+
+# Release notes, written by semantic-release
+CHANGELOG.md

.prettierrc.mjs

@@ -0,0 +1,6 @@
+import config from '@dhis2/config-prettier'
+
+/**
+ * @type {import("prettier").Config}
+ */
+export default { ...config }

ARCHITECTURE.md

@@ -54,16 +54,16 @@ In React fashion we rely on composition to enable the goal of having
 
 This means that:
 
--   complex components are constructed by connecting several simple
-    components,
+- complex components are constructed by connecting several simple
+  components,
 
--   a consumer must have direct access to the building blocks that make up
-    composed components,
+- a consumer must have direct access to the building blocks that make up
+  composed components,
 
--   components need to be isolated from each other with a well defined
-    interface and scope,
+- components need to be isolated from each other with a well defined
+  interface and scope,
 
--   components needs to be reusable across many contexts.
+- components needs to be reusable across many contexts.
 
 > _Note_: Components are published under the `@dhis2-ui` scope, and will
 > eventually be considered part of the external API as things stabilize.
@@ -110,6 +110,6 @@ through that collection.
 
 UI uses two different scopes on NPM to publish:
 
--   `@dhis2/ui*`: Collections and utilities.
+- `@dhis2/ui*`: Collections and utilities.
 
--   `@dhis2-ui/*`: Component packages are published under this scope.
+- `@dhis2-ui/*`: Component packages are published under this scope.