Skip to content

Disable GitHub Pages deploy on every push - #72

Merged
jmjava merged 1 commit into
mainfrom
cursor/disable-pages-on-push-9094
Aug 3, 2026
Merged

Disable GitHub Pages deploy on every push#72
jmjava merged 1 commit into
mainfrom
cursor/disable-pages-on-push-9094

Conversation

@jmjava

@jmjava jmjava commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Summary

Stops Deploy to GitHub Pages from running on pushes to main (including the failing run 30730398390).

Repo Pages is already configured as Deploy from branch → gh-pages, and docs are published with ./scripts/deploy-docs-pages-local.sh. The Actions workflow was still trying to deploy from main on docs/** changes and failing environment protection rules.

Changes

  • .github/workflows/pages.yml: remove the push trigger; keep workflow_dispatch only
  • docs/demos/TOOLING.md: note that Pages CI is not push-triggered

REASONS Canvas

  • Work ID: n/a (ops/chore; no canvas change)
  • Canvas path: n/a

Checklist

  • Canvas updated or synced (n/a)
  • One operation per coding session respected
  • Tests added or updated (n/a — workflow trigger change)
  • Review completed
  • Safeguards respected
  • CI gates pass (validate-command-adapters, test-adapter-install, validate-canvas, validate-diagrams)
  • If command/prompt adapter files changed, ran adapter parity validation (n/a)

Test Plan

  • Confirmed failing run was workflow Deploy to GitHub Pages / .github/workflows/pages.yml on push to main
  • Confirmed repo Pages source is gh-pages branch (legacy), not GitHub Actions
  • After merge, a docs change on main should not start this workflow
  • Manual publish path unchanged: ./scripts/deploy-docs-pages-local.sh
Open in Web Open in Cursor 

Pages is published from the gh-pages branch via the local deploy script; keep the workflow for manual workflow_dispatch only.

Co-authored-by: John Menke <jmjava@gmail.com>
@jmjava
jmjava marked this pull request as ready for review August 3, 2026 18:12
@cursor

cursor Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@jmjava
jmjava merged commit 048d18f into main Aug 3, 2026
5 checks passed
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.

2 participants