docs: fix heading formatting in commands.md and phrasing in tools-api.md#20679
docs: fix heading formatting in commands.md and phrasing in tools-api.md#20679g-samroberts merged 4 commits intogoogle-gemini:mainfrom
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request focuses on improving the clarity and consistency of the project's documentation. It addresses minor formatting issues and refines phrasing in key reference files, making the information more accessible and easier to understand for users and contributors. Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request introduces minor but helpful documentation updates. The changes in commands.md improve heading structure and formatting consistency. The phrasing update in tools-api.md enhances readability. The changes are correct and well-implemented. I have no further suggestions.
|
Hi! Just a heads-up: for a PR to be officially accepted here, it usually needs to be linked to an existing issue. Here is the Contributors Guide. Good luck applying for Gsoc. |
|
Hi @Jomak-x, thank you for the heads-up. So just to make it clear, should I always open an issue first before submitting a PR, even for minor documentation and formatting fixes like this? |
g-samroberts
left a comment
There was a problem hiding this comment.
Looks like you need to resolve conflicts, as tools-api.md has changed. Also, please don't remove the "Build-in Commands" header.
|
|
||
| Slash commands provide meta-level control over the CLI itself. | ||
|
|
||
| ### Built-in Commands |
Add tier 3 header back
docs/reference/tools-api.md
Outdated
| While direct programmatic registration of new tools by users isn't explicitly | ||
| detailed as a primary workflow in the provided files for typical end-users, the | ||
| architecture supports extension through: | ||
| detailed as a primary workflow in this documentation, the architecture supports |
There was a problem hiding this comment.
I think there's a conflict here as this section no longer exists. you can undo this change and it should resolve the conflict.
Undo outdated doc changes to resolve merge conflict
|
Conflicts should be fixed now, thank you for your time! |
….md (google-gemini#20679) Co-authored-by: Sam Roberts <[email protected]>
….md (google-gemini#20679) Co-authored-by: Sam Roberts <[email protected]>
….md (#20679) Co-authored-by: Sam Roberts <[email protected]>
….md (google-gemini#20679) Co-authored-by: Sam Roberts <[email protected]>
….md (google-gemini#20679) Co-authored-by: Sam Roberts <[email protected]>
….md (google-gemini#20679) Co-authored-by: Sam Roberts <[email protected]>
Hi Team! I am an ICT student preparing for GSoC, and I'm excited to start contributing to the Gemini CLI. I am currently going through the documentation to familiarize myself with the architecture, and I found a couple of minor formatting/phrasing inconsistencies that I'd like to clean up.
Summary
Fixed heading hierarchy formatting in
commands.mdand refined phrasing intools-api.md.Details
commands.md: fixed hierarchy and added missing colons to the/chat sharesub-command to match formatting.tools-api.md: Updated phrase under the custom tools section for better readability.Related Issues
#20843
How to Validate
Review the changed lines in
docs/cli/commands.mdanddocs/core/tools-api.mdPre-Merge Checklist