fix: add --legacy-peer-deps to npm install

[email protected].0 requires react@^16.5.1 || ^17.0.0 || ^18.0.0 Project uses [email protected], which is incompatible at the peer level. Use --legacy-peer-deps to allow the installation to proceed.
fix: use origin/main workflow with env vars for registry credentials
2026-09-06 23:52:02 -07:00 · 2026-09-06 23:40:51 -07:00 · 2026-09-06 23:37:56 -07:00 · 2026-09-06 23:34:54 -07:00
1 changed files with 8 additions and 9 deletions
@@ -1,12 +1,14 @@
-name: Build & Push Portfolio Image
+name: CI

 on:
  push:
-    branches:
-      - main
+    branches: [main]
  pull_request:
-    branches:
-      - main
+    branches: [main]
+
+env:
+  REGISTRY: forgejo.riotpiao.com
+  IMAGE: forgejo.riotpiao.com/rock/portfolio

 jobs:
  test:
@@ -20,7 +22,7 @@ jobs:
        uses: actions/checkout@v4

      - name: Install dependencies
-        run: npm ci
+        run: npm install --legacy-peer-deps

      - name: Run tests
        run: npm test -- --run 2>&1 || echo "Tests completed"
@@ -30,9 +32,6 @@ jobs:
    needs: test
    if: github.event_name == 'push' && github.ref == 'refs/heads/main'
    runs-on: node
-    env:
-      REGISTRY: forgejo.riotpiao.com
-      IMAGE: forgejo.riotpiao.com/rock/portfolio
    steps:
      - name: Install Node.js and Docker
        run: |
Author	SHA1	Message	Date
Story Crater Bot	a45fae0287	fix: add --legacy-peer-deps to npm install CI / Test (pull_request) Successful in 1m48s Details CI / Build & Push Image (pull_request) Skipped Details [email protected].0 requires react@^16.5.1 \|\| ^17.0.0 \|\| ^18.0.0 Project uses [email protected], which is incompatible at the peer level. Use --legacy-peer-deps to allow the installation to proceed.	2026-09-06 23:52:02 -07:00
Story Crater Bot	22f5afdb58	fix: use origin/main workflow with env vars for registry credentials CI / Test (pull_request) Failing after 28s Details CI / Build & Push Image (pull_request) Skipped Details Resolved merge conflicts by using current origin/main workflow which: - Includes test job (npm ci + npm test) - Has delete old latest image step - Uses --build-arg for COMMIT_SHA - Uses env vars for registry credentials (FORGEJO_REGISTRY_USER/TOKEN) All features from main preserved, registry login fixed.	2026-09-06 23:40:51 -07:00
Story Crater Bot	b3e5d63e86	fix: use env vars for docker registry credentials	2026-09-06 23:37:56 -07:00
Story Crater Bot	1f66db0ba4	fix: validate registry credentials before docker login Add credential validation step to catch missing secrets early with clear error message. Use direct secret injection (not env vars) for better security. Isolate docker config to /tmp/docker-config.	2026-09-06 23:34:54 -07:00