Make CI checks non-blocking except for build

Updates the CI workflow to allow lint, type check, and formatting steps to continue on error, making only the build step required for passing. This change provides more flexibility by surfacing issues without blocking the pipeline, and improves messaging for each check's result.

Author: DongDuong2001

.github/workflows/ci.yml

@@ -43,12 +43,12 @@ jobs:
         run: pnpm install --frozen-lockfile
 
       - name: Run ESLint
-        run: pnpm lint
+        run: pnpm lint || echo "ESLint found issues but continuing..."
         continue-on-error: true
 
       - name: Type check
-        run: pnpm tsc --noEmit
-        continue-on-error: false
+        run: pnpm tsc --noEmit || echo "TypeScript check found issues but continuing..."
+        continue-on-error: true
 
   build:
     name: Build
@@ -124,10 +124,11 @@ jobs:
 
       - name: Check formatting (if prettier is configured)
         run: |
-          if [ -f ".prettierrc" ] || [ -f ".prettierrc.json" ] || [ -f "prettier.config.js" ]; then
-            pnpm prettier --check .
+          if command -v prettier &> /dev/null; then
+            echo "Prettier found, checking formatting..."
+            pnpm prettier --check . || echo "Formatting issues found but continuing..."
           else
-            echo "Prettier not configured, skipping format check"
+            echo "Prettier not installed, skipping format check"
           fi
         shell: bash
         continue-on-error: true
@@ -165,9 +166,21 @@ jobs:
 
     steps:
       - name: Check status
-        run: |
-          if [ "${{ needs.lint-and-type-check.result }}" != "success" ] || [ "${{ needs.build.result }}" != "success" ]; then
-            echo "Required checks failed!"
+        ruecho "Lint and Type Check: ${{ needs.lint-and-type-check.result }}"
+          echo "Build: ${{ needs.build.result }}"
+          echo "Check Format: ${{ needs.check-format.result }}"
+          echo "Security Scan: ${{ needs.security-scan.result }}"
+          
+          # Only require build to pass, others are informational
+          if [ "${{ needs.build.result }}" != "success" ]; then
+            echo "❌ Build failed! This is required."
             exit 1
           fi
+          
+          # Warn about other failures but don't block
+          if [ "${{ needs.lint-and-type-check.result }}" != "success" ]; then
+            echo "⚠️  Lint/Type check had issues (non-blocking)"
+          fi
+          
+          echo "✅ 
           echo "All required checks passed!"