Stop vendoring brew.sh theme#2219
Merged
Merged
Conversation
- Use the shared `brew.sh` theme assets by URL to avoid duplicated files. - Keep only the formulae layouts and includes that this site needs. - Drop feed and blog-only files because formulae pages do not publish posts. - Remove `jekyll-remote-theme` so builds do not fetch a remote theme. - Refresh the lockfile for the simplified Jekyll dependency set.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR de-vendors the brew.sh theme from formulae.brew.sh by switching to shared theme assets hosted on https://brew.sh, and removes blog/feed-related layouts/includes that aren’t used by the formulae site. It also updates the lockfile to reflect the simplified Jekyll dependency set.
Changes:
- Load shared
brew.shCSS and image assets via absolute URLs and update the CSP allowlist accordingly. - Remove vendored theme assets (SCSS + SVGs) and drop unused blog/feed layouts/includes.
- Refresh
Gemfile.lock(notablyio-eventpatch update).
Reviewed changes
Copilot reviewed 9 out of 28 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| Gemfile.lock | Lockfile refresh (includes io-event patch bump). |
| assets/img/linuxbrew.svg | Remove vendored theme image. |
| assets/img/homebrew.svg | Remove vendored theme image. |
| assets/img/homebrew-social-card.svg | Remove vendored theme image. |
| assets/img/forkme_right_gray_6d6d6d.svg | Remove vendored theme image. |
| assets/img/brewtestbot.svg | Remove vendored theme image. |
| assets/img/brewsci.svg | Remove vendored theme image. |
| assets/img/brew.svg | Remove vendored theme image. |
| assets/img/apple-touch-icon.svg | Remove vendored theme image. |
| assets/css/style.scss | Remove vendored theme stylesheet. |
| _sass/pygments.scss | Remove vendored Pygments styles (theme-provided styling now expected externally). |
| _layouts/post.html | Remove blog post layout (not used by formulae site). |
| _layouts/index.html | Remove homepage/blog-oriented layout (not used by formulae site). |
| _layouts/home.html | Remove blog/home wrapper layout (not used by formulae site). |
| _layouts/base.html | Point CSS/icons/logo/fork ribbon at brew.sh and adjust CSP; remove feed injection. |
| _includes/newsletter.html | Remove newsletter embed include (blog-only). |
| _includes/feed.html | Remove feed meta include. |
| _config.yml | Switch default SEO image and logo to brew.sh hosted assets. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
p-linnane
approved these changes
Jun 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
brew.shtheme assets by URL to avoid duplicated files.jekyll-remote-themeso builds do not fetch a remote theme.