-
-
Notifications
You must be signed in to change notification settings - Fork 12.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chainhook 1.5.1 (new formula) #168028
chainhook 1.5.1 (new formula) #168028
Conversation
e334b5c
to
586a85d
Compare
Thanks for contributing to Homebrew! 🎉 It looks like you're having trouble with a CI failure. See our contribution guide for help. You may be most interested in the section on dealing with CI failures. You can find the CI logs in the Checks tab of your pull request. |
|
98ca9c1
to
7ba9ec4
Compare
@chenrui333 @SMillerDev I have addressed your comments! I thought I fixed all of the commit messages to match the expected style, but it still is failing CI on that point. |
@chenrui333 Could I get a re-review on this? |
@MicaiahReid one last comment, can you also squash all your commits as well? And add this formula into autobump.txt file. Thanks! |
355741d
to
ec9bc08
Compare
19fb5af
to
9a1fbb8
Compare
@chenrui333 Thanks for the feedbacks! I think this should be good to go 🚀 |
@MicaiahReid, thanks for your first contribution to homebrew-core!! 🎉 🥇 Without awesome contributors like you, it would be impossible to maintain Homebrew to the high level of quality users have come to expect. Thank you!!!! |
HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew test <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingHOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
)? If this is a new formula, does it passbrew audit --new <formula>
?