docs: fix typo in extensions command in README#3097
docs: fix typo in extensions command in README#3097NaniToka wants to merge 2 commits intogoogleapis:mainfrom
Conversation
### 🛠️ Fix typo in Gemini CLI section This pull request fixes a typo in the **Gemini CLI** section of the README. * Corrected `/exttensions list` → `/extensions list` ### ✅ Why this change? The incorrect command may confuse users when trying to list extensions in the CLI, leading to errors during usage. ### 📌 Scope * Documentation update only * No code changes ### 🔗 Related Issue Closes googleapis#3096
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request modifies the README.md to update the list of supported databases. However, the change introduces a formatting regression by removing spaces after commas, which negatively impacts readability. The reviewer noted that this modification appears to be an accidental change that does not address the intended typo fix described in the pull request's metadata and provided a suggestion to restore the correct formatting.
NaniToka
left a comment
NaniToka
left a comment
There was a problem hiding this comment.
🧾 Review for PR #3097
👍 Overall feedback
The intent of this PR is correct — it fixes a typo in the Gemini CLI command and improves documentation clarity. However, the current change introduces an unrelated formatting issue in the same section.
🐛 Issue identified
The modification to the “Other Databases” line removes spaces after commas:
Snowflake,Trino,and more.
This reduces readability and is inconsistent with the rest of the documentation style.
💡 Suggested fix
Please restore proper spacing for readability:
- Snowflake,Trino,and more.
- Snowflake, Trino, and more.
⚠️ Additional note
The PR description mentions fixing:
/exttensions list → /extensions list
However, that change does not appear in the current diff. Please ensure the intended fix is included in the commit.
✅ Recommendation
Fix formatting regression
Ensure typo fix is included in this PR
Then re-request review
✔️ Final verdict
Changes required before merge
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
|
Closing as a duplicate of #3096 (review) |
2 participants
🛠️ Fix typo in Gemini CLI section
This pull request fixes a typo in the Gemini CLI section of the README.
/exttensions list→/extensions list✅ Why this change?
The incorrect command may confuse users when trying to list extensions in the CLI, leading to errors during usage.
📌 Scope
🔗 Related Issue
Closes #3096
Description
Fixes a typo in the Gemini CLI command within the README file. This ensures users can correctly list installed extensions without encountering command errors.
PR Checklist