Skip to content
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

Extend README with brew installation instruction [no ci] #7616

Merged
merged 1 commit into from
May 30, 2024

Conversation

makuche
Copy link
Contributor

@makuche makuche commented May 29, 2024

llama.cpp is now available in Homebrew (see here and here). I think it would be helpful to extend the README to raise awareness of that installation possibility. This PR is adding that information to the README file.

I am not sure if this information should be positioned somewhere else in the README, maybe right before the Prepare and Quantize chapter or even before the Get the Code chapter, I am happy to receive feedback on that as well.

@mofosyne
Copy link
Collaborator

dont forget to add [no ci] to documentation only changes in your commit title

@mofosyne mofosyne added the Review Complexity : Low Trivial changes to code that most beginner devs (or those who want a break) can tackle. e.g. UI fix label May 29, 2024
@makuche makuche changed the title Extend README with brew installation instruction Extend README with brew installation instruction [no ci] May 29, 2024
Copy link
Collaborator

@mofosyne mofosyne left a comment

Choose a reason for hiding this comment

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

@mofosyne mofosyne added documentation Improvements or additions to documentation merge ready indicates that this may be ready to merge soon and is just holding out in case of objections labels May 29, 2024
@mofosyne mofosyne merged commit 2e2340d into ggerganov:master May 30, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation merge ready indicates that this may be ready to merge soon and is just holding out in case of objections Review Complexity : Low Trivial changes to code that most beginner devs (or those who want a break) can tackle. e.g. UI fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants