Skip to content

feat: add brewfile validation - #1132

Merged
inffy merged 7 commits into
mainfrom
validate-brew
Oct 22, 2025
Merged

feat: add brewfile validation#1132
inffy merged 7 commits into
mainfrom
validate-brew

Conversation

@inffy

@inffy inffy commented Oct 22, 2025

Copy link
Copy Markdown
Member

These are picked from Bluefin:
ublue-os/bluefin#3450
ublue-os/bluefin#3449

We already had the brewfiles on the image but i feel it's easier to find now that they are in the root brew folder and then copied to the correct folder.

Comment thread .github/workflows/validate-brewfiles.yml Fixed
@inffy
inffy marked this pull request as ready for review October 22, 2025 07:50
@inffy
inffy requested a review from NiHaiden as a code owner October 22, 2025 07:50
Copilot AI review requested due to automatic review settings October 22, 2025 07:50
@inffy
inffy enabled auto-merge October 22, 2025 07:50

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 adds Homebrew brewfile validation to the Aurora build system. The changes introduce a GitHub Actions workflow to validate brewfiles on pull requests and reorganize how brewfiles are handled during the container build process.

Key Changes:

  • Adds automated brewfile validation via GitHub Actions
  • Moves brewfile copying to the shared build stage for better organization
  • Removes redundant tap declaration from aurora-ai.Brewfile

Reviewed Changes

Copilot reviewed 4 out of 7 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/validate-brewfiles.yml New workflow to validate brewfiles using Homebrew bundle
build_files/shared/build-base.sh Copies brewfiles to /usr/share/ublue-os/homebrew/ during build
Containerfile Adds /brew directory to build context
brew/aurora-ai.Brewfile Removes redundant homebrew/cask tap declaration

Comment thread .github/workflows/validate-brewfiles.yml
Comment thread .github/workflows/validate-brewfiles.yml Outdated
Co-authored-by: renner <80410025+renner0e@users.noreply.github.com>
@inffy
inffy added this pull request to the merge queue Oct 22, 2025
Merged via the queue into main with commit 9d740ee Oct 22, 2025
29 checks passed
@inffy
inffy deleted the validate-brew branch October 22, 2025 16:47
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.

4 participants