devops: Move PHP build to Taskfile (#3164)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

.github/workflows/test-php.yaml

@@ -23,16 +23,8 @@ jobs:
     steps:
       - name: 📂 Checkout code
         uses: actions/checkout@v3
-      - uses: actions-rs/toolchain@v1
+      - uses: arduino/setup-task@v1
         with:
-          toolchain: stable
-      - run: sh build.sh
-        working-directory: bindings/prql-php
-      - name: 📦 Install dependencies using Composer
-        uses: php-actions/composer@v6
-        with:
-          args: --working-dir=bindings/prql-php
-          php_extensions: FFI
-      - name: 🧪 Run tests using PHPUnit
-        run: vendor/bin/phpunit tests
-        working-directory: bindings/prql-php
+          repo-token: "${{ secrets.GITHUB_TOKEN }}"
+      - run: task build-php
+      - run: task test-php

Taskfile.yml

@@ -327,6 +327,18 @@ tasks:
       - package.json
     generates:
       - node_modules/*
+
+  build-php:
+    - cargo build -p prql-lib --release
+    - mkdir -p bindings/prql-php/lib/
+    - cp target/release/libprql_lib.* bindings/prql-lib/libprql_lib.h
+      bindings/prql-php/lib/
+    - cd bindings/prql-php && composer install
+
+  test-php:
+    dir: bindings/prql-php
+    cmds:
+      - vendor/bin/phpunit tests
 # The next three tasks are not used for either:
 #  - the Dockerfile (installing brew takes forever)
 #    so the Dockerfile simply installs hugo & nodejs directly

bindings/prql-php/README.md

@@ -49,17 +49,17 @@ This will pull-in ext-ffi extension, because it's declared in `composer.json`.
 
 ### Building
 
-There is a `build.sh` script that:
+There is a `task build-php` script that:
 
 - runs cargo to build `libprql_lib`,
-- copies `libprql_lib.so` into `lib`,
+- copies `libprql_lib.*` into `lib`,
 - copies `libprql_lib.h` into `lib`.
 
 ### Tests
 
 ```
-sh build.sh
-./vendor/bin/phpunit tests
+task build-php
+task test-php
 ```
 
 ### Code style