Add Ollama installation to the setup script - #16
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bf9efe2b35
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| # 14. Ollama — local models for the Severed Floor | ||
| if ! command -v ollama &>/dev/null; then | ||
| echo "🦙 Ollama not found. The work needs a local model stack. Installing..." | ||
| curl -fsSL https://ollama.com/install.sh | sh |
There was a problem hiding this comment.
Use a macOS-compatible Ollama installer
When Ollama is absent on the macOS systems this project targets—including the macos-latest E2E job—the upstream install.sh rejects non-Linux hosts with “This script is intended to run on Linux only.” Because pipefail and set -e are enabled, this command terminates the entire setup without installing Ollama or printing the completion message; use Ollama's macOS package/cask instead.
Useful? React with 👍 / 👎.
…ilure (#19) * Initial plan * fix: use brew install ollama instead of curl installer to avoid GUI launch failure in CI Co-authored-by: bobbravo2 <348865+bobbravo2@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: bobbravo2 <348865+bobbravo2@users.noreply.github.com>
Ollama is now provisioned alongside the rest of the macOS environment so local model workflows are available out of the box. The setup remains idempotent and skips installation when
ollamais already present.Install flow
install.shollamais missingCoverage
Docs / CI
ollama --versionto macOS CI verification