-
Notifications
You must be signed in to change notification settings - Fork 79
tested out paths locally and updated readme.md #186
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
@suryansh00001 Can you sign the Google CLA so that we can go ahead with review? Or else I can close this and pick up the changes myself. Thanks |
![]() @mihirvala-crestdata It says "It looks like you've already signed this CLA. If you'd like to edit your contact information, you may do so below."
|
CI/Check is successful now. It needed time to reflect the change, I guess. |
@mihirvala-crestdata Now is it good to go? |
@mihirvala-crestdata Do I need to do something else to close this issue? |
@mihirvala-crestdata I'm extremely sorry for those issues. I'm new to open source contributions. Kindly check out the changes I made now... I guess there won't be any issue now. I'll take care from now onwards |
@mihirvala-crestdata Sir, is it okay now? |
@mihirvala-crestdata hello sir , Please check out the changes I made.. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one final change for Claude Code settings location in Windows.
Rest looks good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done the requested changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Verified Client Config locations.
@dandye Can review once? so that we can have this merged. |
Documented where Gemini CLI, Claude Desktop, Claude Code, Cursor IDE, VS Code, and Cline expect their JSON config file
Closed #185