Skip to content

Conversation

@babblebey
Copy link
Contributor

@babblebey babblebey commented Aug 30, 2025

Description

This pull request makes a minor update to the installation verification steps in the Ollama guide. The change improves the order of commands to check that Ollama is installed before starting the service.

  • Installation verification: Moved the ollama --version command to immediately follow the installation step, ensuring users verify the installation before proceeding to start the Ollama service. (docs/guides/ollama-guide.mdx)

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-general-review or @continue-detailed-review

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]


Summary by cubic

Improved the Ollama install guide by verifying installation before starting the service. Moved ollama --version directly after the install step and removed the duplicate check.

@babblebey babblebey requested a review from a team as a code owner August 30, 2025 21:35
@babblebey babblebey requested review from Patrick-Erichsen and removed request for a team August 30, 2025 21:35
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 30, 2025
@github-actions
Copy link

github-actions bot commented Aug 30, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Aug 31, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 31, 2025
@bdougie
Copy link
Collaborator

bdougie commented Aug 31, 2025

Thanks for the commit. Can you confirm the CLA @babblebey ?

@babblebey
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@Patrick-Erichsen Patrick-Erichsen merged commit 4cec16a into continuedev:main Sep 3, 2025
94 of 117 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Sep 3, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Sep 3, 2025
@sestinj
Copy link
Contributor

sestinj commented Sep 3, 2025

🎉 This PR is included in version 1.13.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Sep 3, 2025

🎉 This PR is included in version 1.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm This PR has been approved by a maintainer released size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants