-
Notifications
You must be signed in to change notification settings - Fork 61.2k
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
Add mention of JetBrains IDEs in changing AI models #36656
base: main
Are you sure you want to change the base?
Conversation
How to review these changes 👓Thank you for your contribution. To review these changes, choose one of the following options: A Hubber will need to deploy your changes internally to review. Table of review linksThe table shows the files in the Key: fpt: Free, Pro, Team; ghec: GitHub Enterprise Cloud; ghes: GitHub Enterprise Server 🤖 This comment is automatically generated. |
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.
PR Overview
This PR adds support for JetBrains IDEs in the documentation for changing and using AI models for GitHub Copilot Chat. The changes update headings, add new bullet items for JetBrains IDEs, and refine instruction notes for improved clarity and consistency.
- Added JetBrains IDEs as a supported client in multiple documentation files.
- Updated titles and instructional blocks for clarity.
- Included a new note on multi-model support in a public preview phase.
Reviewed Changes
File | Description |
---|---|
content/copilot/using-github-copilot/ai-models/changing-the-ai-model-for-copilot-chat.md | Updated headings and instructional notes, including the addition of a note for multi-model support and changes to the dropdown description. |
content/copilot/using-github-copilot/ai-models/using-gemini-flash-in-github-copilot.md | Added a bullet item for JetBrains IDEs. |
content/copilot/using-github-copilot/ai-models/using-claude-sonnet-in-github-copilot.md | Added duplicate bullet items for JetBrains IDEs which need consolidation. |
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (1)
content/copilot/using-github-copilot/ai-models/changing-the-ai-model-for-copilot-chat.md:165
- [nitpick] The label 'CURRENT-MODEL' may be unclear to users; consider using a more descriptive label or a variable substitution consistent with other sections.
1. In the bottom right of the chat view, select an AI model of your choice from the **CURRENT-MODEL** {% octicon "chevron-down" aria-hidden="true" %} dropdown menu.
content/copilot/using-github-copilot/ai-models/using-claude-sonnet-in-github-copilot.md
Show resolved
Hide resolved
|
||
To use multi-model {% data variables.product.prodname_copilot_chat_short %}, you must use {% data variables.product.prodname_vs %} 2022 version 17.12 or later. See the [{% data variables.product.prodname_vs %} downloads page](https://visualstudio.microsoft.com/downloads/). | ||
> [!IMPORTANT] To use multi-model {% data variables.product.prodname_copilot_chat_short %}, you must use {% data variables.product.prodname_vs %} 2022 version 17.12 or later. See the [{% data variables.product.prodname_vs %} downloads page](https://visualstudio.microsoft.com/downloads/). |
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.
Let me know if I shouldn’t add it, but it seems crucial because the article all about it.
@@ -159,6 +162,6 @@ These instructions are for the JetBrains IDEs. For instructions on different cli | |||
|
|||
1. Click the **{% octicon "copilot" aria-hidden="true" %}** icon in the status bar. | |||
1. In the popup menu, click **Open {% data variables.product.prodname_copilot_chat %}**. | |||
1. In the bottom right of the chat view, select an AI model of your choice from the {% octicon "chevron-down" aria-hidden="true" %} dropdown menu. | |||
1. In the bottom right of the chat view, select an AI model of your choice from the **CURRENT-MODEL** {% octicon "chevron-down" aria-hidden="true" %} dropdown menu. |
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.
@@ -135,11 +135,14 @@ To use multi-model {% data variables.product.prodname_copilot_chat_short %}, you | |||
|
|||
{% jetbrains %} | |||
|
|||
> [!NOTE] Multiple model support in {% data variables.product.prodname_copilot_chat_short %} is in {% data variables.release-phases.public_preview %} and is subject to change. |
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.
Should we make it reusable data? It's repeated 4 times on the page.
Although there are a few other sentences that are also repeated, but they're not reusable data, so it's probably worth leaving as is.
And a side note: I couldn't find any mention of Claude 3.7 Sonnet Thinking, probably need to raise an issue for it. |
@@ -22,12 +22,14 @@ redirect_from: | |||
* {% data variables.product.prodname_copilot_chat_short %} in {% data variables.product.prodname_vscode %} | |||
* {% data variables.product.prodname_copilot_chat_short %} in {% data variables.product.prodname_vs %} 2022 version 17.13 or later | |||
* Immersive mode in {% data variables.product.prodname_copilot_chat_short %} in {% data variables.product.github %} | |||
* {% data variables.product.prodname_copilot_chat_short %} in JetBrains IDEs |
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.
Why don't you have a variable with the product name for JetBrains IDEs? It seems like it's used a lot in the code.
Thanks for raising this, @rakleed! We'll review it as soon as we can |
@@ -159,6 +162,6 @@ These instructions are for the JetBrains IDEs. For instructions on different cli | |||
|
|||
1. Click the **{% octicon "copilot" aria-hidden="true" %}** icon in the status bar. |
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.
Xxx |
Why:
Closes: #36305
What's being changed (if available, include any code snippets, screenshots, or gifs):
Add mention of JetBrains IDEs in changing AI models, also add absent text and unify it
Check off the following: