[PR #5655] [MERGED] remove template #58567

Closed
opened 2026-04-29 13:27:22 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/5655
Author: @pdevine
Created: 7/12/2024
Status: Merged
Merged: 7/14/2024
Merged by: @pdevine

Base: mainHead: pdevine/remove-template


📝 Commits (2)

📊 Changes

5 files changed (+16 additions, -50 deletions)

View changed files

📝 api/types.go (+2 -0)
📝 cmd/cmd.go (+0 -2)
📝 cmd/interactive.go (+14 -38)
📝 cmd/interactive_test.go (+0 -3)
📝 server/routes.go (+0 -7)

📄 Description

Remove the broken /set template command in the CLI. This is an alternative to #5613 that doesn't add another parameter to the /api/chat endpoint.

Given the /set template command was broken for 6 months and only one person noticed (thank you @protosam) I think it's probably safe to remove this.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ollama/ollama/pull/5655 **Author:** [@pdevine](https://github.com/pdevine) **Created:** 7/12/2024 **Status:** ✅ Merged **Merged:** 7/14/2024 **Merged by:** [@pdevine](https://github.com/pdevine) **Base:** `main` ← **Head:** `pdevine/remove-template` --- ### 📝 Commits (2) - [`623533f`](https://github.com/ollama/ollama/commit/623533f1ce19900d6b50b9a701698b57593eb0df) remove template - [`c76c71e`](https://github.com/ollama/ollama/commit/c76c71e82a8770e00e40c290223b3e9b756f399f) remove template from tests ### 📊 Changes **5 files changed** (+16 additions, -50 deletions) <details> <summary>View changed files</summary> 📝 `api/types.go` (+2 -0) 📝 `cmd/cmd.go` (+0 -2) 📝 `cmd/interactive.go` (+14 -38) 📝 `cmd/interactive_test.go` (+0 -3) 📝 `server/routes.go` (+0 -7) </details> ### 📄 Description Remove the broken `/set template` command in the CLI. This is an alternative to #5613 that doesn't add another parameter to the `/api/chat` endpoint. Given the `/set template` command was broken for 6 months and only one person noticed (thank you @protosam) I think it's probably safe to remove this. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-29 13:27:22 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/ollama#58567