Split CI workflows (#349)

- split CI workflow into three pieces `lint`/`frontend`/`backend` to run
workflows only when needed
- run tests only on `pull_request` events
- add dedicated build step in frontend workflow
- update `pre-commit` hooks version to be up-to-date

---------

Co-authored-by: Auguste Baum <auguste@probabl.ai>

.github/workflows/backend.yml

@@ -0,0 +1,53 @@
+name: Backend
+
+on:
+  pull_request:
+    types: [synchronize]
+    paths:
+      - 'src/**'
+      - 'tests/**'
+      - pyproject.toml
+      - Makefile
+      - requirements*.txt
+      - .github/workflows/backend.yml
+
+permissions:
+  contents: read
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-python@v5
+        with:
+          python-version: "3.12"
+      - name: Lint
+        run: |
+          python -m pip install --upgrade pip
+          python -m pip install --upgrade pre-commit
+
+          pre-commit run --all-files ruff
+
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-python@v5
+        with:
+          python-version: "3.12"
+      - name: Build frontend and share library
+        shell: bash
+        run: make build-frontend
+      - name: Build package distributions
+        run: |
+          python -m pip install --upgrade pip
+          python -m pip install --upgrade build
+          python -m build
+      - name: Install
+        run: |
+          python -m pip install dist/*.whl --no-dependencies
+          python -m pip install -r requirements.txt -r requirements-test.txt
+      - name: Pytest
+        timeout-minutes: 5
+        run: python -m pytest

.github/workflows/frontend.yml

@@ -0,0 +1,62 @@
+name: Frontend
+
+on:
+  pull_request:
+    types: [synchronize]
+    paths:
+      - 'frontend/**'
+      - Makefile
+      - .github/workflows/lint-frontend.yml
+
+permissions:
+  contents: read
+  pull-requests: write
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: "20"
+      - name: Lint
+        run: |
+          cd frontend
+
+          npm install
+          npm run type-check
+          npm run lint
+          npm run format
+          npm run style-lint
+
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: "20"
+      - name: Test
+        run: |
+          cd frontend
+          npm install
+          npm run test:unit:coverage
+      - name: Report coverage
+        if: always()
+        uses: davelosert/vitest-coverage-report-action@v2
+        with:
+          working-directory: ./frontend
+          pr-number: ${{ github.event.number }}
+
+  build:
+    runs-on: ubuntu-latest
+    needs: test
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: "20"
+      - name: Build
+        shell: bash
+        run: make build-frontend

.github/workflows/lint.yml

@@ -0,0 +1,28 @@
+name: Lint
+
+on: [pull_request]
+
+permissions:
+  contents: read
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-python@v5
+        with:
+          python-version: "3.12"
+      - name: Lint
+        run: |
+          python -m pip install --upgrade pip
+          python -m pip install --upgrade pre-commit
+
+          pre-commit run --all-files check-yaml
+          pre-commit run --all-files check-toml
+          pre-commit run --all-files check-added-large-files
+          pre-commit run --all-files check-merge-conflict
+          pre-commit run --all-files detect-private-key
+          pre-commit run --all-files end-of-file-fixer
+          pre-commit run --all-files trailing-whitespace
+          pre-commit run --all-files typos

.github/workflows/pull-request.yml

@@ -2,7 +2,6 @@ on:
   pull_request:
     types: [opened]
 
-
 jobs:
   add-assignee:
     runs-on: ubuntu-latest

.pre-commit-config.yaml

@@ -1,8 +1,6 @@
-# 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: v3.2.0
+    rev: v4.6.0
     hooks:
       - id: check-yaml
       - id: check-toml
@@ -13,20 +11,18 @@ repos:
       - id: trailing-whitespace
 
   - repo: https://github.com/crate-ci/typos
-    rev: v1.21.0
+    rev: v1.24.6
     hooks:
       - id: typos
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    # Ruff version.
-    rev: v0.3.7
+    rev: v0.6.5
     hooks:
-      # Run the linter.
       - id: ruff
-        # Comment to disable autofix:
+        files: ^(src|tests)/
         args: [--fix]
-      # Run the formatter.
       - id: ruff-format
+        files: ^(src|tests)/
 
   - repo: local
     hooks:

Makefile

@@ -45,9 +45,3 @@ build-frontend:
 	cp -a frontend/dist/. src/skore/ui/static
 	# clean up
 	rm -rf frontend/dist
-
-build-doc:
-	python -m pip install -e . -r requirements-doc.txt
-	cd doc
-	# Run a make instruction inside the doc folder
-	$(MAKE) -C doc html

VERSION.txt

@@ -1 +1 @@
-0.1.0-rc.1
+0.0.0

frontend/package.json

@@ -5,13 +5,12 @@
   "type": "module",
   "scripts": {
     "dev": "vite --host",
-    "build": "npm-run-all --parallel type-check \"build-only {@}\" --",
+    "build": "vite build",
     "build:lib": "vite build -c vite.config.lib.ts",
     "preview": "vite preview",
     "test:unit": "vitest run",
     "test:unit:watch": "vitest",
     "test:unit:coverage": "vitest run --coverage",
-    "build-only": "vite build",
     "type-check": "vue-tsc --build --force",
     "lint": "eslint . --ext .vue,.js,.jsx,.cjs,.mjs,.ts,.tsx,.cts,.mts --fix --ignore-path .gitignore",
     "style-lint": "stylelint --fix src/**/*.{css,vue}",