-
-
Notifications
You must be signed in to change notification settings - Fork 12.5k
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
ente-cli 0.2.2 (new formula) #201919
base: master
Are you sure you want to change the base?
ente-cli 0.2.2 (new formula) #201919
Conversation
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. |
c493376
to
71160dc
Compare
The Linux test does not pass, and I can't explain why. I don't have any Linux with homebrew available to test it right now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your submission, @mathieutu . I left some comments and please take a look.
@daeho-ro Thanks for your help. I've made all the changes, except the test ones: Hence (except if you have any idea):
|
That's fine, we can wait for the next release. |
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>
?Hey,
First new formula here, to add the ente cli tool to Homebrew.
Fix ente-io/ente#706
The
brew test
does not pass yet (it will in the next release), because of an issue unrelated to the formula (keychain error, that does not happen when executing directly the command). The formula is tested and used in local (macOS arm64).Thanks,
Mathieu