Merge pull request #84 from peterlau123/fix/jekyll-build-and-link-checker

fix(ci): Fix GitHub Pages Jekyll build and link checker

Author: peterlau123

.github/workflows/jekyll.yml

@@ -24,7 +24,7 @@ jobs:
       - name: Setup Ruby
         uses: ruby/setup-ruby@v1
         with:
-          ruby-version: '3.3'
+          ruby-version: "3.3"
           bundler-cache: true
 
       - name: Setup Pages
@@ -36,6 +36,16 @@ jobs:
         env:
           JEKYLL_ENV: production
 
+      - name: Verify build output
+        run: |
+          echo "Checking _site/index.html..."
+          if grep -q "{% for post" _site/index.html; then
+            echo "ERROR: Jekyll template code found in _site/index.html - Jekyll build failed!"
+            head -20 _site/index.html
+            exit 1
+          fi
+          echo "Build verification passed - _site/index.html is properly rendered"
+
       - name: Upload artifact
         uses: actions/upload-pages-artifact@v4
 

.github/workflows/site-validation.yml

@@ -6,7 +6,7 @@ on:
   pull_request:
   schedule:
     # Run weekly on Monday at 00:00 UTC
-    - cron: '0 0 * * 1'
+    - cron: "0 0 * * 1"
   workflow_dispatch:
 
 jobs:
@@ -19,7 +19,7 @@ jobs:
       - name: Setup Ruby
         uses: ruby/setup-ruby@v1
         with:
-          ruby-version: '3.3'
+          ruby-version: "3.3"
           bundler-cache: true
 
       - name: Build site
@@ -28,7 +28,7 @@ jobs:
       - name: Setup Python
         uses: actions/setup-python@v6
         with:
-          python-version: '3.x'
+          python-version: "3.x"
 
       - name: Install html5validator
         run: pip install html5validator
@@ -38,10 +38,34 @@ jobs:
           html5validator --root _site/ --also-check-css --ignore '*/css/bootstrap.*' --ignore '*/css/bootstrap.min.*' --format text || echo "HTML validation completed with warnings"
         continue-on-error: true
 
+      - name: Serve site locally
+        run: |
+          python3 -m http.server 8080 --directory _site &
+          echo "SERVER_PID=$!" >> $GITHUB_ENV
+          sleep 2
+
       - name: Check links
         uses: lycheeverse/lychee-action@v2
         with:
-          args: --base file://${{ github.workspace }}/_site --verbose --no-progress --exclude 'https://peterlau123\.github\.io.*' './_site/**/*.html'
+          args: |
+            --base http://localhost:8080
+            --verbose
+            --no-progress
+            --timeout 30
+            --retry 2
+            --exclude 'https://peterlau123\.github\.io.*'
+            --exclude 'http://huangxuan\.me'
+            --exclude 'http://cdn\.bootcss\.com'
+            --exclude 'https://oss\.maxcdn\.com'
+            --exclude 'https://ghbtns\.com'
+            --exclude 'https://pagead2\.googlesyndication\.com'
+            --exclude-mail
+            --accept 200,204,301,302,307,308,403
+            'http://localhost:8080'
           fail: true
         env:
           GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
+
+      - name: Stop local server
+        if: always()
+        run: kill $SERVER_PID || true

.gitignore

@@ -4,11 +4,11 @@ _archive
 node_modules
 .vscode
 *.log
-*.lock
+# *.lock  # Commented out to allow Gemfile.lock for Jekyll dependency management
 *.sh
 .DS_Store
 .jekyll-cache
 */.DS_Store
 */*/.DS_Store
 .idea
-_site/
+_site/

.jekyll-metadata

@@ -0,0 +1,76 @@
+GEM
+  remote: https://rubygems.org/
+  specs:
+    addressable (2.8.7)
+      public_suffix (>= 2.0.2, < 7.0)
+    colorator (1.1.0)
+    concurrent-ruby (1.3.5)
+    em-websocket (0.5.3)
+      eventmachine (>= 0.12.9)
+      http_parser.rb (~> 0)
+    eventmachine (1.2.7)
+    ffi (1.17.2)
+    forwardable-extended (2.6.0)
+    google-protobuf (3.23.4)
+    http_parser.rb (0.8.0)
+    i18n (1.14.7)
+      concurrent-ruby (~> 1.0)
+    jekyll (4.3.4)
+      addressable (~> 2.4)
+      colorator (~> 1.0)
+      em-websocket (~> 0.5)
+      i18n (~> 1.0)
+      jekyll-sass-converter (>= 2.0, < 4.0)
+      jekyll-watch (~> 2.0)
+      kramdown (~> 2.3, >= 2.3.1)
+      kramdown-parser-gfm (~> 1.0)
+      liquid (~> 4.0)
+      mercenary (>= 0.3.6, < 0.5)
+      pathutil (~> 0.9)
+      rouge (>= 3.0, < 5.0)
+      safe_yaml (~> 1.0)
+      terminal-table (>= 1.8, < 4.0)
+      webrick (~> 1.7)
+    jekyll-paginate (1.1.0)
+    jekyll-sass-converter (3.0.0)
+      sass-embedded (~> 1.54)
+    jekyll-watch (2.2.1)
+      listen (~> 3.0)
+    kramdown (2.5.1)
+      rexml (>= 3.3.9)
+    kramdown-parser-gfm (1.1.0)
+      kramdown (~> 2.0)
+    liquid (4.0.4)
+    listen (3.9.0)
+      rb-fsevent (~> 0.10, >= 0.10.3)
+      rb-inotify (~> 0.9, >= 0.9.10)
+    mercenary (0.4.0)
+    pathutil (0.16.2)
+      forwardable-extended (~> 2.6)
+    public_suffix (5.1.1)
+    rake (13.3.0)
+    rb-fsevent (0.11.2)
+    rb-inotify (0.11.1)
+      ffi (~> 1.0)
+    rexml (3.4.2)
+    rouge (3.30.0)
+    safe_yaml (1.0.5)
+    sass-embedded (1.58.3)
+      google-protobuf (~> 3.21)
+      rake (>= 10.0.0)
+    terminal-table (3.0.2)
+      unicode-display_width (>= 1.1.1, < 3)
+    unicode-display_width (2.6.0)
+    webrick (1.9.1)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  jekyll (~> 4.0)
+  jekyll-paginate
+  rake
+  webrick (~> 1.7)
+
+BUNDLED WITH
+   1.17.2