Skip to content

Conversation

vaygr
Copy link
Contributor

@vaygr vaygr commented Sep 27, 2025

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

@github-actions github-actions bot added new formula PR adds a new formula to Homebrew/homebrew-core swift Swift use is a significant feature of the PR or issue macos-only Formula depends on macOS labels Sep 27, 2025
@botantony
Copy link
Contributor

You can fix incorrect style error by adding the formula to not_a_binary_url_prefix_allowlist

@vaygr vaygr force-pushed the utiluti-new-formula branch from 9ac72ee to 66c4adf Compare September 27, 2025 14:36
@github-actions github-actions bot added the automerge-skip `brew pr-automerge` will skip this pull request label Sep 27, 2025

uses_from_macos "swift" => :build, since: :sonoma # swift 6.0+

patch :DATA
Copy link
Member

Choose a reason for hiding this comment

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

can this patch be upstreamed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@chenrui333 yes, I already submitted it, however I'm not sure when the next release is expected.

Copy link
Member

Choose a reason for hiding this comment

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

can you add a code comment to call out the upstream issue/pr?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@chenrui333 fixed in 6a2357d.

@vaygr vaygr force-pushed the utiluti-new-formula branch from 66c4adf to 4cbf440 Compare September 28, 2025 04:05
@vaygr vaygr force-pushed the utiluti-new-formula branch from 4cbf440 to 6a2357d Compare September 28, 2025 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge-skip `brew pr-automerge` will skip this pull request macos-only Formula depends on macOS new formula PR adds a new formula to Homebrew/homebrew-core swift Swift use is a significant feature of the PR or issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants