Skip to content

Comments

Purge CDN selectively#2932

Merged
JacobCoffee merged 2 commits intomainfrom
purge-smart
Feb 21, 2026
Merged

Purge CDN selectively#2932
JacobCoffee merged 2 commits intomainfrom
purge-smart

Conversation

@JacobCoffee
Copy link
Member

Description

  • Since we reorganized we can more easily tell which templates were changed and where we need to purge.

Copilot AI review requested due to automatic review settings February 21, 2026 19:25
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 907515a219

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Copy link

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 enhances the Fastly CDN cache purging workflow to selectively purge only the affected parts of the site based on which files were changed, rather than always purging the entire application cache. The workflow now analyzes git diffs to determine whether to purge the main app cache or specific app-level caches (like downloads, sponsors, jobs, etc.).

Changes:

  • Added checkout step to analyze changed files in push events
  • Implemented logic to resolve which cache keys to purge based on changed file paths
  • Modified purge step to support multiple cache keys in a loop

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Only purge section-specific keys when app templates/static change,
fall back to full site purge for global template/static changes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@JacobCoffee
Copy link
Member Author

All three review comments addressed in 60d913d:

  • Uses github.event.before..github.sha for full push range diff
  • Updated to actions/checkout@v6
  • Condition now uses github.event_name == 'push'

@JacobCoffee JacobCoffee merged commit 7d86d4c into main Feb 21, 2026
14 checks passed
@JacobCoffee JacobCoffee deleted the purge-smart branch February 21, 2026 19:49
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