fix: ditch test in docker publish + add inputs for helm

Signed-off-by: WoodenMaiden <yann.pomie@laposte.net>
WoodenMaiden · Feb 29, 2024 · ecd3450 · ecd3450
1 parent 3ca7af8
commit ecd3450
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 32 deletions.
diff --git a/.github/workflows/cd.yaml b/.github/workflows/cd.yaml
@@ -21,7 +21,7 @@ jobs:
             const no_v = tag.replace('v', '')
             core.setOutput('no_v', no_v)
 
-  build-publish:
+  publish_docker:
     runs-on: ubuntu-latest
     needs: set_vars
 
@@ -32,26 +32,6 @@ jobs:
     steps:
       - uses: actions/checkout@v3
 
-      - name: Setup node js
-        uses: actions/setup-node@v3.2.0
-
-      - name: Use Node.js 16
-        uses: actions/setup-node@v3
-        with:
-          node-version: 16
-          cache: 'npm'
-
-      - name: Cache
-        uses: actions/cache@v3.0.2
-        with:
-          path: ~/.npm
-          key: npm-${{ hashFiles('package-lock.json') }}
-          restore-keys: npm-
-
-      - run: npm ci --ignore-scripts
-      - run: npm run build --if-present
-      - run: npm test
-
       - name: Login to ghcr.io
         uses: docker/login-action@v2
         with:
@@ -69,14 +49,6 @@ jobs:
           push: true
           tags: ghcr.io/woodenmaiden/relfinderreformedapi:${{ needs.set_vars.outputs.no_v }},ghcr.io/woodenmaiden/relfinderreformedapi:latest
 
-      - name: Test Report
-        uses: dorny/test-reporter@v1
-        if: success() || failure() # run this step even if previous step failed
-        with:
-          name: JEST Tests # Name of the check run which will be created
-          path: reports/jest*.xml # Path to test results
-          reporter: jest-junit # Format of test results
-
   helm_release:
     permissions:
       contents: write
@@ -102,3 +74,5 @@ jobs:
           CR_TOKEN: '${{ secrets.GITHUB_TOKEN }}'
         with:
           mark_as_latest: true
+          skip_existing: true
+          charts_dir: charts
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -18,17 +18,17 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [16.x, 18.x]
+        node-version: [16.x, 18.x, 20.x]
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
 
     steps:
       - uses: actions/checkout@v3
 
       - name: Setup node js
-        uses: actions/setup-node@v3.2.0
+        uses: actions/setup-node@v4.0.2
 
       - name: Use Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@v3
+        uses: actions/setup-node@v4.0.2
         with:
           node-version: ${{ matrix.node-version }}
           cache: 'npm'