Aay/fix/feedback2#3
Open
alpaycb wants to merge 9 commits into
Open
Conversation
Updated installation instructions for Couchbase Shell on different platforms and added details for creating documents in Couchbase.
Updated the README to improve clarity and correctness in instructions for installing and configuring Couchbase Shell, including changes to command descriptions and examples.
Updated API key references and instructions for clarity and consistency. Added alternative API key options and adjusted configuration steps. Google's Gemini/AIStudio API Key instructions were added to make the workshop available without relying on a paid subscription.
Expanded the Couchbase section to include key features and benefits for RAG applications.
Updated the README to clarify workshop details and improve formatting.
Updated the Readme.md to clarify the integration of LLMs and Couchbase Shell, enhancing explanations and instructions throughout the document.
Updated the Couchbase features and benefits for Retrieval-Augmented Generation (RAG) to enhance clarity and detail.
ldoguin
reviewed
Nov 25, 2025
ldoguin
left a comment
ldoguin
left a comment
Contributor
There was a problem hiding this comment.
just a couple of minor issues
| # Register your cluster | ||
| ( clusters get $cluster_name | cb-env register $cluster_name $in."connection string" | ||
| --capella-organization "yourOrgIdentifier" | ||
| clusters get $cluster_name | cb-env register $cluster_name $in."connection string" |
Contributor
There was a problem hiding this comment.
@alpaycb I think we need to keep the opening and closing parenthesis as it's what allows multiline to work in nushell. Without it we can't copy and paste this.
|
|
||
| Let's add some documents about Couchbase features: | ||
|
|
||
| - Now let's create our first document on Couchbase Platform changes, which are coming with v8.0. These are very new information, for which LLM's referenced embedding model was trained yet |
Contributor
There was a problem hiding this comment.
I am assuming you meant 'was not trained yet'
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
I've adjusted the content according to the experiences from our Berlin meetup and from my SE-Team's feedback,
Thanks...