Skip to content

ci: retire obsolete GitHub Pages deploy#3

Merged
zelinewang merged 2 commits into
mainfrom
fix/retire-github-pages-workflow
Jul 13, 2026
Merged

ci: retire obsolete GitHub Pages deploy#3
zelinewang merged 2 commits into
mainfrom
fix/retire-github-pages-workflow

Conversation

@zelinewang

@zelinewang zelinewang commented Jul 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove the obsolete GitHub Pages deployment workflow
  • replace it with a credential-free static validation workflow
  • keep Netlify as the only supported deployment target
  • leave the accepted FIRMS runtime correction and netlify.toml unchanged

Root cause

Default-branch pushes still ran Deploy to GitHub Pages, but Pages is not enabled for this repository. The failure is deterministic and unrelated to the application diff:

Both fail in actions/configure-pages@v4 with enablement: false, Get Pages site failed, and HttpError: Not Found. Since README and runtime configuration identify Netlify as the supported host, enabling a second deployment target would be misleading.

CI discovery

Deleting the final workflow correctly retired Pages, but the repository CodeQL default setup still analyzes the actions language. The first PR run then failed with CodeQL could not process any code written in GitHub Actions (exit 32). Rather than mutate repository security settings or restore a deployment, this PR adds Validate static app: a read-only workflow that runs only node --check src/app.js and python3 -m py_compile scripts/*.py on PRs and main pushes. The rerun is green.

Exact diff

A .github/workflows/ci.yml
D .github/workflows/deploy.yml
2 files changed, 22 insertions(+), 37 deletions(-)

Verification

  • branch created from current origin/main (157155e)
  • no tracked workflow references github-pages, configure-pages, or deploy-pages
  • replacement workflow has contents: read only and performs no deployment
  • local node --check src/app.js
  • local python3 -m py_compile scripts/*.py
  • workflow YAML parsed successfully (actionlint is unavailable locally)
  • git diff --check origin/main...HEAD
  • public sweep: 0 Layer 1-4 blockers (4 informational/adoption warnings)
  • netlify.toml unchanged: SHA-256 9a5268cc2922ff4f29dd8aca809be87a33186727f3952b4b2af43ac24ade684d
  • src/app.js unchanged: SHA-256 937ae88ee72035d52ad7e4a1c314ce15126e8b5112fd2b033ceb5a3b6c1cae97
  • PR checks: 8 successful, 1 skipped, 0 failing/pending; replacement validate, CodeQL Actions/JS/Python, and Netlify checks all pass

Rollback

Revert commits e80fcaa and 0f123a5 if GitHub Pages is intentionally enabled and adopted later. This PR does not change Pages or Netlify settings and remains unmerged.

@netlify

netlify Bot commented Jul 13, 2026

Copy link
Copy Markdown

Deploy Preview for firesight ready!

Name Link
🔨 Latest commit 0f123a5
🔍 Latest deploy log https://app.netlify.com/projects/firesight/deploys/6a5440c3fbb2d30009cbf752
😎 Deploy Preview https://deploy-preview-3--firesight.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@zelinewang
zelinewang merged commit 0ad4efb into main Jul 13, 2026
9 checks passed
@zelinewang
zelinewang deleted the fix/retire-github-pages-workflow branch July 13, 2026 01:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant