Skip to content

Conversation

b-reich
Copy link
Contributor

@b-reich b-reich commented Aug 22, 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>?

Without tree there is a bug in ansible-role-init

❯ ansible-role-init test-role
Ansible Role : test-role Creating....
- Role test-role was created successfully
ansible-role-init:4: command not found: tree

I would say it could be recommended but its not allowed in hombrew-core

@github-actions github-actions bot added python Python use is a significant feature of the PR or issue rust Rust use is a significant feature of the PR or issue labels Aug 22, 2025
@chenrui333 chenrui333 added the CI-no-bottles Merge without publishing bottles label Aug 22, 2025
@chenrui333
Copy link
Member

make sense, thanks @b-reich!

@chenrui333 chenrui333 added this pull request to the merge queue Aug 22, 2025
Merged via the queue into Homebrew:main with commit 96d41b3 Aug 22, 2025
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-no-bottles Merge without publishing bottles python Python use is a significant feature of the PR or issue rust Rust use is a significant feature of the PR or issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants