Skip to content

ci(api-docs): prune superseded search indexes on deploy#265

Draft
twishabansal wants to merge 1 commit into
mainfrom
chore/prune-search-index-on-deploy
Draft

ci(api-docs): prune superseded search indexes on deploy#265
twishabansal wants to merge 1 commit into
mainfrom
chore/prune-search-index-on-deploy

Conversation

@twishabansal

Copy link
Copy Markdown
Contributor

The deploy overlaid the build onto gh-pages additively (keep_files: true), so each rebuild's new content-hashed offline-search-index..json was added while the previous one lingered forever. Replace the peaceiris step with a single-commit clone+overlay (the pattern api-docs-backfill.yml already uses) that drops the old search index in each rebuilt directory before copying. CNAME and .nojekyll are preserved explicitly.

@twishabansal twishabansal requested a review from a team as a code owner June 8, 2026 10:26
@twishabansal twishabansal marked this pull request as draft June 8, 2026 10:26
@twishabansal twishabansal added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 8, 2026
The deploy overlaid the build onto gh-pages additively (keep_files: true),
so each rebuild's new content-hashed offline-search-index.<hash>.json was
added while the previous one lingered forever. Replace the peaceiris step
with a single-commit clone+overlay (the pattern api-docs-backfill.yml
already uses) that drops the old search index in each rebuilt directory
before copying. CNAME and .nojekyll are preserved explicitly.
@anubhav756 anubhav756 force-pushed the chore/prune-search-index-on-deploy branch from c47e382 to 331823b Compare June 11, 2026 10:00
@anubhav756 anubhav756 added the priority: p2 Moderately-important priority. Fix may not be included in next release. label Jun 11, 2026
@anubhav756 anubhav756 removed their assignment Jun 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Indicates a pull request not ready for merge, due to either quality or timing. priority: p2 Moderately-important priority. Fix may not be included in next release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants