Skip to content

Add Ollama installation to the setup script - #16

Merged
bobbravo2 merged 3 commits into
mainfrom
copilot/add-ollama-installation
Jul 28, 2026
Merged

Add Ollama installation to the setup script#16
bobbravo2 merged 3 commits into
mainfrom
copilot/add-ollama-installation

Conversation

Copilot AI commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

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 ollama is already present.

  • Install flow

    • Added an Ollama section to install.sh
    • Uses the upstream installer when ollama is missing
    • Preserves the existing skip-on-present pattern
  • Coverage

    • Added Bats coverage for both install and skip paths
    • Updated the all-tools-present scenario to include Ollama
  • Docs / CI

    • Documented Ollama in the README tool table
    • Added ollama --version to macOS CI verification
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
else
  echo "🦙 Ollama is already running on the Severed Floor."
fi

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 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".

Comment thread install.sh Outdated
# 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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge 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>
@bobbravo2
bobbravo2 merged commit e8131e5 into main Jul 28, 2026
2 checks passed
@bobbravo2
bobbravo2 deleted the copilot/add-ollama-installation branch July 28, 2026 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants