-
Notifications
You must be signed in to change notification settings - Fork 751
docs: using GH Copilot for AI Assist endpoint #5035
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
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Pull Request Overview
This PR updates the documentation for configuring and using AI completions with marimo by adding a GitHub Copilot section and standardizing the configuration file path references. Key changes include:
- Introducing a dedicated GitHub Copilot section with setup instructions.
- Updating configuration file paths to use $XDG_CONFIG_HOME/marimo/marimo.toml.
- Standardizing UI configuration hints across various guides.
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| docs/guides/editor_features/ai_completion.md | Added GitHub Copilot instructions and updated configuration hints. |
| docs/guides/configuration/index.md | Updated configuration file path to $XDG_CONFIG_HOME/marimo/marimo.toml. |
| docs/getting_started/quickstart.md | Updated configuration file path to $XDG_CONFIG_HOME/marimo/marimo.toml. |
Comments suppressed due to low confidence (1)
docs/guides/configuration/index.md:26
- Ensure that the change in the configuration file path is also documented in the release notes or migration guide to help users transition smoothly.
in a "$XDG_CONFIG_HOME/marimo/marimo.toml" file.
| 1. Install openai: `pip install openai` | ||
|
|
||
| 2. Add the following to your `marimo.toml`: | ||
| 2. Add the following to your `marimo.toml` (or configure in the UI settings in the editor): |
Copilot
AI
May 22, 2025
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.
For clarity and consistency, consider adding a brief note explaining why the alternative UI configuration exists and when it should be used.
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.
LGTM
|
🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.13.12-dev16 |
Fixes #4305