Skip to content

Clean PR preview sites from gh-pages branch history#6161

Merged
jamadeo merged 3 commits intomainfrom
jackamadeo/clean-gh-pages
Dec 18, 2025
Merged

Clean PR preview sites from gh-pages branch history#6161
jamadeo merged 3 commits intomainfrom
jackamadeo/clean-gh-pages

Conversation

@jamadeo
Copy link
Collaborator

@jamadeo jamadeo commented Dec 17, 2025

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR implements automated cleanup of the gh-pages branch history to remove preview sites from closed PRs. The cleanup process uses git-filter-repo to rewrite the branch history, removing directories that only contain hidden files or are empty. The concurrency model is changed from per-PR to global to prevent race conditions during history rewriting operations.

Key Changes

  • Added scripts/clean-gh-pages.sh to identify and remove PR preview directories with only hidden files or empty directories
  • Modified workflow concurrency from preview-${{ github.ref }} to pr-preview to serialize all workflow executions
  • Added new cleanup job that runs when PRs are closed to rewrite gh-pages branch history

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
scripts/clean-gh-pages.sh New script that identifies empty or hidden-file-only PR preview directories and uses git-filter-repo to remove them from branch history
.github/workflows/pr-website-preview.yml Updates concurrency model, pins action versions, and adds cleanup job to rewrite gh-pages history when PRs close

@jamadeo jamadeo merged commit 7ff3adc into main Dec 18, 2025
39 checks passed
@jamadeo jamadeo deleted the jackamadeo/clean-gh-pages branch December 18, 2025 21:22
zanesq added a commit that referenced this pull request Dec 18, 2025
* 'main' of github.com:block/goose: (28 commits)
  Clean PR preview sites from gh-pages branch history (#6161)
  fix: make goose reviewer less sycophantic (#6171)
  revert /reply to previous behavior (replacing session history) when full conversation provided (#6058)
  chore: manually update version (#6166)
  Integrate pricing with canonical model (#6130)
  Regenerate canonical models when release branch is created. (#6127)
  fix: use correct parameter name in read_module handler (#6148)
  docs: blog for code mode MCP (#6126)
  test: add ACP integration test (#6150)
  docs: auto download updates (#6163)
  fix: respect default_enabled value of platform extensions (#6159)
  docs: skills (#6062)
  fix: add conditional configuration for GOOSE_BIN_DIR in PATH (#5940)
  Update dependencies to help in Fedora packaging (#5835)
  fix: make goose reviewer less bad (#6154)
  docs: create/edit recipe button (#6145)
  fix(google): Fix 400 Bad Request error with Gemini 3 thought signatures (#6035)
  fix: we don't need to warn about tool count when in code mode (#6149)
  deps: upgrade agent-client-protocol to 0.9.0 (#6109)
  fix(providers): fix for gemini-cli on windows to work around cmd's multiline prompt limitations #5911 (#5966)
  ...

# Conflicts:
#	ui/desktop/src/api/sdk.gen.ts
#	ui/desktop/src/hooks/useAgent.ts
bkvarda pushed a commit to bkvarda/goose that referenced this pull request Dec 19, 2025
Signed-off-by: Brandon Kvarda <brandon.kvarda@databricks.com>
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