clarity on how to add openrouter for llm-support #1509
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
May I suggest a separate PR for the formatting changes? |
This reverts commit 59c56e9.
|
@jordigh updated per your comment! |
lshalon
force-pushed
the
readme-claude-and-other-support
branch
from
c7c291d to
0d0b3f2
Compare
paulfitz
reviewed
Mar 13, 2025
There was a problem hiding this comment.
Thanks for this @lshalon ! It reads well and looks helpful. Thanks for tweaking to point out it isn't the only way to use Claude. My one remaining qualm is that it still reads like a pitch for OpenRouter, and once landed, it will sound like that pitch is coming from Grist maintainers - and we really don't have an opinion on OpenRouter one way or another. Could you, just after the "## Using Grist ..." and a little "(Instructions contributed by @lshalon)" preamble? With that context for readers, I am comfortable landing it and grateful for your work in prepping it.
lshalon
force-pushed
the
readme-claude-and-other-support
branch
from
0d0b3f2 to
6738fd0
Compare
|
@paulfitz Thanks! Let me know how it looks now? |
paulfitz
approved these changes
Mar 13, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context
This PR updates the README to include documentation for OpenRouter integration, which allows users to access various LLM providers through a single API interface.
Proposed solution
Added clear documentation in the README explaining how to set up and use OpenRouter as an alternative LLM provider. This includes configuration steps, API key setup, and examples of how to select different models through the OpenRouter interface.
Related issues
N/A
Has this been tested?
👍