chore: improve build.yml

.github/workflows/build.yml

@@ -13,17 +13,14 @@ jobs:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        os: [ubuntu-22.04, macos-11, windows-2019]
         include:
           - os: ubuntu-22.04
-            dockerfile: ./npm/linux-x64/Dockerfile
-            platform: linux-x64
-          - os: ubuntu-22.04
-            dockerfile: ./npm/linuxmusl-x64/Dockerfile
             platform: linuxmusl-x64
+          - os: ubuntu-22.04
+            platform: linux-x64
           - os: macos-11
             platform: darwin-x64
-          - os: windows-2019
+          - os: windows-2020
             platform: windows-x64
 
     steps:
@@ -42,9 +39,9 @@ jobs:
       run: mkdir release
 
     - name: Build with Docker (${{ matrix.platform }})
-      if: contains(matrix.os, 'ubuntu') && matrix.dockerfile != ''
+      if: contains(matrix.os, 'ubuntu')
       run: |
-        docker build -t ${{ matrix.platform }} -f ${{ matrix.dockerfile }} .
+        docker build --build-arg PLATFORM=${{ matrix.platform }} --build-arg LIBHEIF_V=v1.17.6 -t test . -f ./platform/Dockerfile
         docker run --name ${{ matrix.platform }} ${{ matrix.platform }}
         docker cp ${{ matrix.platform }}:/heif-converter/src/build/Release/converter.node ./release/converter.${{ matrix.platform }}.node
         docker container rm ${{ matrix.platform }}
@@ -63,10 +60,12 @@ jobs:
     #     cp ./src/build/Release/converter.node ./src/build/Release/converter-windows-x64.node
 
     - name: Create tarball of the release folder
-      run: tar -czf ${{ matrix.platform }}.tar.gz -C release .
+      run: |
+        cd ./release
+        tar -czf ${{ matrix.platform }}.tar.gz *
 
     - name: Upload artifact
       uses: actions/upload-artifact@v2
       with:
         name: release
-        path: ${{ matrix.platform }}.tar.gz
+        path: ./release/${{ matrix.platform }}.tar.gz

package.json

@@ -26,7 +26,8 @@
   "types": "index.d.ts",
   "main": "index.js",
   "scripts": {
-    "test": "node --test test/test.spec.js"
+    "test": "node --test test/test.spec.js",
+    "build": "node-gyp configure --directory=src && node-gyp build --directory=src"
   },
   "publishConfig": {
     "@myunisoft:registry": "https://registry.npmjs.org/"

platform/Dockerfile

@@ -0,0 +1,10 @@
+ARG PLATFORM
+ARG LIBHEIF_V
+FROM myunisoft91/heif-converter.$PLATFORM:$LIBHEIF_V
+LABEL maintainer="Nicolas Marteau <[email protected]>"
+
+WORKDIR heif-converter
+COPY . .
+RUN npm install && \
+    npm run build && \
+    npm run test