From 9bf09b05dfa62a0716d58e9229ef8d4c8b888126 Mon Sep 17 00:00:00 2001 From: Ludovic Henry Date: Tue, 8 Sep 2026 00:33:23 +0200 Subject: [PATCH 1/3] djade: add build-djade.yml for riscv64 wheels Rust/PyO3-maturin (bindings = "bin") Django template formatter CLI. No pyo3 dependency and no ABI tag, so one native riscv64gc-unknown-linux-gnu build in the manylinux_2_39 container covers every interpreter; tested against README-documented formatting rules (stdin round-trip, --check, endblock labelling) since the sdist ships no wheel-level test suite. --- .github/workflows/build-djade.yml | 123 ++++++++++++++++++++++++++++++ 1 file changed, 123 insertions(+) create mode 100644 .github/workflows/build-djade.yml diff --git a/.github/workflows/build-djade.yml b/.github/workflows/build-djade.yml new file mode 100644 index 000000000..4ac6d97cd --- /dev/null +++ b/.github/workflows/build-djade.yml @@ -0,0 +1,123 @@ +# SPDX-FileCopyrightText: 2026 The RISE Project +# SPDX-License-Identifier: MIT +--- +# This workflow is based on the `build` job of +# https://github.com/adamchainz/djade/blob/1.9.0/.github/workflows/main.yml +name: Build djade wheels (riscv64) + +on: + workflow_dispatch: + inputs: + version: + description: 'djade version to build (git tag, e.g. 1.9.0)' + required: true + default: '1.9.0' + pull_request: + paths: + - '.github/workflows/build-djade.yml' + +concurrency: + group: ${{ github.workflow }}-${{ inputs.version || '1.9.0' }}-${{ github.head_ref || github.run_id }} + cancel-in-progress: true + +permissions: + contents: read # to fetch code (actions/checkout) + +env: + # `inputs.version` is empty on pull_request events; default to 1.9.0 there. + DJADE_VERSION: ${{ inputs.version || '1.9.0' }} + +jobs: + setup: + uses: $/.github/workflows/_setup.yml + + build_wheel: + needs: [setup] + name: Build djade ${{ inputs.version || '1.9.0' }} manylinux_riscv64 + runs-on: ubuntu-24.04-riscv + timeout-minutes: 360 + + steps: + - name: Checkout djade v${{ env.DJADE_VERSION }} + uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1 + with: + repository: adamchainz/djade + ref: ${{ env.DJADE_VERSION }} + persist-credentials: false + + # `[tool.maturin] bindings = "bin"`: the wheel is one compiled executable + # with no ABI tag, so a single native build covers every interpreter. + - name: Build wheel + uses: PyO3/maturin-action@e83996d129638aa358a18fbd1dfb82f0b0fb5d3b # v1.51.0 + with: + command: build + target: riscv64gc-unknown-linux-gnu + args: --release --locked --out dist + manylinux: '2_39' + before-script-linux: git config --global --add safe.directory "*" + + - uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1 + with: + name: djade-${{ env.DJADE_VERSION }}-manylinux_riscv64 + path: dist/*.whl + if-no-files-found: error + + test_wheel: + name: Test djade ${{ inputs.version || '1.9.0' }} on Python ${{ matrix.python-version }} + needs: [setup, build_wheel] + runs-on: ubuntu-24.04-riscv + timeout-minutes: 30 + env: + # Without this uv would reuse the runner image's system CPython for 3.12 + # and download a standalone build for the others. + UV_PYTHON_PREFERENCE: only-managed + strategy: + fail-fast: false + matrix: + # This repo's default interpreter matrix (gotcha in workflow-anatomy.md); + # the wheel is interpreter-agnostic (bindings = "bin"), so every + # interpreter -- including free-threaded -- exercises the same binary. + python-version: ['3.12', '3.13', '3.14', '3.14t'] + + steps: + - name: Download wheel + uses: actions/download-artifact@3e5f45b2cfb9172054b4087a40e8e0b5a5461e7c # v8.0.1 + with: + name: djade-${{ env.DJADE_VERSION }}-manylinux_riscv64 + + - name: Install Python + uses: astral-sh/setup-uv@20cfd1bf945f4377ade1205e4dbc17946fc9a30d # v10.0.1 + with: + python-version: ${{ matrix.python-version }} + activate-environment: true + enable-cache: false + + - name: Install wheel + run: uv pip install --reinstall --no-index --find-links . djade + + # No wheel-level test suite ships in the sdist (gotcha 187): djade's own + # tests/integration.rs suite is run by upstream's own `cargo test`, never + # packaged. Drive the real formatter against the README's own documented + # rules instead of settling for a bare --help check: stdin/stdout + # reformatting, already-formatted detection, --check mode, and the + # endblock-labelling rule applied to a file in place. + - name: Test wheel + run: | + set -euo pipefail + djade --help >/dev/null + + out=$(printf '{{ engine | paint }}' | djade -) + [ "$out" = '{{ engine|paint }}' ] + + printf '{{ engine|paint }}\n' | djade - >/dev/null + + set +e + printf '{{ engine | paint }}' | djade --check - >/dev/null + rc=$? + set -e + [ "$rc" -eq 1 ] + + work=$(mktemp -d) + printf '{%% block funnel %%}\nhi\n{%% endblock %%}\n' > "$work/t.html" + djade "$work/t.html" + grep -qx '{% endblock funnel %}' "$work/t.html" From 52a37e1efd3b2d0de330bde6c62c0742bfca1acb Mon Sep 17 00:00:00 2001 From: Ludovic Henry Date: Tue, 8 Sep 2026 00:48:32 +0200 Subject: [PATCH 2/3] build-djade: expect exit code 1 when djade reformats a real file Reformatting a file in place (non-stdin, non-check mode) exits 1, verified against upstream's own test_stdin_mixed_with_files assertion; the earlier step ran under 'set -e' and aborted before the grep ever ran. --- .github/workflows/build-djade.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/workflows/build-djade.yml b/.github/workflows/build-djade.yml index 4ac6d97cd..0b5f9a0ca 100644 --- a/.github/workflows/build-djade.yml +++ b/.github/workflows/build-djade.yml @@ -119,5 +119,9 @@ jobs: work=$(mktemp -d) printf '{%% block funnel %%}\nhi\n{%% endblock %%}\n' > "$work/t.html" + set +e djade "$work/t.html" + rc=$? + set -e + [ "$rc" -eq 1 ] grep -qx '{% endblock funnel %}' "$work/t.html" From 6898d723366426e18cd4cb218498fd17f655ad6f Mon Sep 17 00:00:00 2001 From: Ludovic Henry Date: Tue, 8 Sep 2026 01:20:46 +0200 Subject: [PATCH 3/3] build-djade: add missing publish job The publish job was dropped when the workflow was first authored; wire it the same way as build-ripgrep.yml/build-prek.yml so the wheel actually reaches pypi.riseproject.dev once merged to main. --- .github/workflows/build-djade.yml | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/.github/workflows/build-djade.yml b/.github/workflows/build-djade.yml index 0b5f9a0ca..8034eb842 100644 --- a/.github/workflows/build-djade.yml +++ b/.github/workflows/build-djade.yml @@ -125,3 +125,13 @@ jobs: set -e [ "$rc" -eq 1 ] grep -qx '{% endblock funnel %}' "$work/t.html" + + publish: + name: Publish djade ${{ inputs.version || '1.9.0' }} + needs: [setup, build_wheel, test_wheel] + permissions: + contents: write + pull-requests: write + uses: $/.github/workflows/_publish-wheel.yml + with: + artifact-pattern: djade-${{ inputs.version || '1.9.0' }}-manylinux_riscv64