-
Notifications
You must be signed in to change notification settings - Fork 2.3k
hotfix: changed website theme #6335
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
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.
Important
Looks good to me! 👍
Reviewed everything up to 0d51a13 in 1 minute and 49 seconds. Click for details.
- Reviewed
329lines of code in4files - Skipped
0files when reviewing. - Skipped posting
4draft 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. website/astro.config.mjs:4
- Draft comment:
Remove the unused import 'starlightThemeNext' if the new theme (rapide) is in use. - Reason this comment was not posted:
Comment was on unchanged code.
2. website/package.json:20
- Draft comment:
Confirm that the theme dependency is updated to 'starlight-theme-rapide' and any reference to the old theme has been removed. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =0%<= threshold50%This comment is asking the PR author to confirm that a dependency update has been fully applied, which is not allowed according to the rules. It is not making a specific code suggestion or pointing out a specific issue with the code.
3. website/src/content/docs/index.mdx:28
- Draft comment:
Verify that the cover image path '../../assets/jan_loaded.png' correctly resolves from this MDX file. - Reason this comment was not posted:
Comment was not on a location in the diff, so it can't be submitted as a review comment.
4. website/src/content/docs/index.mdx:160
- Draft comment:
Ensure all relative links and anchors in the FAQ and Quick Start sections resolve correctly in production. - 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 links resolve correctly, which falls under the category of asking the author to double-check things. It doesn't provide a specific suggestion or point out a specific issue with the code.
Workflow ID: wflow_ragtyiOUxl7LawkR
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.
Important
Looks good to me! 👍
Reviewed 77877ec in 1 minute and 4 seconds. Click for details.
- Reviewed
29lines of code in2files - Skipped
0files when reviewing. - Skipped posting
2draft 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. website/astro.config.mjs:4
- Draft comment:
Removed import of starlightThemeNext. Confirm that it's no longer used anywhere and that starlightThemeRapide fully replaces it. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
2. website/src/content.config.ts:8
- Draft comment:
Removed videosSchema extension from docsSchema. Verify that video schema validations are no longer needed and that this change is intentional. - 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 their intention, which is against the rules. It does not provide a specific suggestion or point out a clear issue with the code. Therefore, it should be removed.
Workflow ID: wflow_JTa3LwHAeDb5p9p5
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
|
Preview URL Astro Docs: https://57a89726.astro-docs-14t.pages.dev |
Small change to the docs site theme.
Important
Switches website theme to
starlight-theme-rapideand updates FAQ formatting inindex.mdx.starlight-theme-nexttostarlight-theme-rapideinastro.config.mjs.starlight-theme-nextand related dependencies frombun.lockandpackage.json.index.mdx.This description was created by
for 77877ec. You can customize this summary. It will automatically update as commits are pushed.