-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Documentation Updates for v0.6.5 #5799
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
|
Preview URL: https://3deee7f6.docs-9ba.pages.dev |
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.
Caution
Changes requested ❌
Reviewed everything up to 2f77c6b in 2 minutes and 56 seconds. Click for details.
- Reviewed
2795lines of code in24files - Skipped
128files when reviewing. - Skipped posting
7draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. docs/src/pages/docs/llama-cpp.mdx:67
- Draft comment:
Ensure backend variant names and CUDA version details remain up-to-date with the latest supported hardware. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =0%<= threshold50%This comment is asking the PR author to ensure that certain details remain up-to-date, which falls under the rule of not asking the author to ensure things. It doesn't provide a specific suggestion or point out a specific issue with the code.
2. docs/theme.config.tsx:76
- Draft comment:
Double-check navigation labels: 'Jan Local Server' vs. 'Jan API Platform'. Confirm that the intended links are correctly assigned. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =0%<= threshold50%The comment is asking the PR author to confirm their intention regarding navigation labels, which violates the rule against asking for confirmation of intention. It does not provide a specific code suggestion or ask for a specific test to be written.
3. docs/src/pages/local-server/troubleshooting.mdx:268
- Draft comment:
Verify that the permission-fix command examples work across platforms; consider testing or adding platform-specific notes. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =0%<= threshold50%The comment is asking the PR author to verify and test the command examples across platforms, which violates the rule against asking the author to ensure the change is tested or to double-check things. It doesn't provide a specific suggestion or point out a specific issue with the code.
4. docs/src/pages/local-server/tabby.mdx:41
- Draft comment:
Clarify if extra configuration is required for connecting Jan with Tabby beyond the steps shown. - Reason this comment was not posted:
Confidence changes required:50%<= threshold50%None
5. docs/src/pages/changelog/2025-07-17-responsive-ui.mdx:21
- Draft comment:
Typographical error: The phrase "can causes" should be corrected to "can cause". - Reason this comment was not posted:
Marked as duplicate.
6. docs/src/pages/docs/mcp-examples/data-analysis/e2b.mdx:148
- Draft comment:
Typo: The phrase "we're to get this fixed" might be clearer as "we'll get this fixed". Consider revising for clarity. - Reason this comment was not posted:
Decided after close inspection that this draft comment was likely wrong and/or not actionable: usefulness confidence = 10% vs. threshold = 50% While this is technically correct and the text is in a changed file (new file), our rules state that we should not make purely informative comments. The meaning is clear despite the minor grammatical error, and this kind of small typo doesn't impact code functionality or documentation clarity in a meaningful way. The grammar error could potentially confuse non-native English speakers. The comment does provide a clear fix. While true, the meaning is still understandable and this is the kind of minor issue that clutters PR reviews without adding significant value. Delete the comment as it's too minor and doesn't affect understanding or functionality significantly.
7. docs/src/pages/local-server/settings.mdx:56
- Draft comment:
Consider checking the file name in the image reference: 'trouble-shooting-04.png' might be intended to be 'troubleshooting-04.png' for consistency with the term used elsewhere. - Reason this comment was not posted:
Decided after close inspection that this draft comment was likely wrong and/or not actionable: usefulness confidence = 20% vs. threshold = 50% While the comment points out a real inconsistency in spelling between the image name and the guide name, this is a relatively minor issue. The image will still work either way. The rules say not to make comments that are obvious or unimportant. This feels like a minor stylistic suggestion rather than a substantive issue requiring a code change. The inconsistency could cause confusion or maintenance issues down the line. Having consistent naming conventions is a good software practice. While consistency is good, this is too minor of an issue to warrant a PR comment. The image works fine either way, and this kind of minor cleanup can be handled in a separate maintenance PR if desired. Delete this comment as it points out a minor stylistic inconsistency that doesn't materially impact the functionality or maintainability of the code.
Workflow ID: wflow_LOx7OIOidrayIAhc
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
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
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Documentation Updates for v0.6.5
Summary
Key Changes
Improvements
Important
Documentation updates for v0.6.5 include new tutorials, reorganized structure, and improved readability, along with a new changelog entry.
/local-server/.Jan Local ServerandMCP Examples.2025-07-17-responsive-ui.mdxwith v0.6.5 release notes.E2B Code SandboxandExa SearchMCP tutorials with setup instructions and examples.llama-cpp.mdxandmodel-parameters.mdxfor clearer language and better accessibility.troubleshooting.mdxwith detailed steps for common issues.This description was created by
for 2f77c6b. You can customize this summary. It will automatically update as commits are pushed.