Skip to content

Conversation

@bkrabach
Copy link

@bkrabach bkrabach commented Apr 18, 2025

Enable the configuration of the Ollama base URL through an environment variable, providing flexibility for different deployment scenarios, such as a remote-hosted Ollama server.

@github-actions
Copy link

github-actions bot commented Apr 18, 2025

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

@bkrabach
Copy link
Author

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

@bkrabach
Copy link
Author

bkrabach commented Apr 18, 2025

@ymichael it appears the cla-signatures branch is not set up on your fork, so it will not pass the CLA Assistant action? Would you be willing to make the requested change to support reading the environment variable for setting the Ollama endpoint? I appreciate the work you did to provide Ollama support in general, this change will let folks use it when running their Ollama service on another host.

@bkrabach bkrabach marked this pull request as ready for review April 18, 2025 12:27
@ymichael
Copy link
Owner

thanks! (separately I removed the CLA check from this repo here: 783bca0)

@ymichael ymichael merged commit 6f8686e into ymichael:main Apr 18, 2025
1 check failed
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