i18n(fr): add package-catalog/studiocms-plugins/studiocms-wysiwyg#124
Conversation
WalkthroughA new French documentation file was added to the package catalog, providing information about the Changes
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
src/content/docs/fr/package-catalog/studiocms-plugins/studiocms-wysiwyg.mdx (1)
22-25: Cohérence des guillemets dans le composant MDX
Dans<PackageManagers pkg="studiocms" type='run' args='add @studiocms/wysiwyg' />, il vaudrait mieux unifier l’usage des guillemets :- <PackageManagers pkg="studiocms" type='run' args='add @studiocms/wysiwyg' /> + <PackageManagers pkg="studiocms" type="run" args="add @studiocms/wysiwyg" />
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/content/docs/fr/package-catalog/studiocms-plugins/studiocms-wysiwyg.mdx(1 hunks)
🧰 Additional context used
🪛 LanguageTool
src/content/docs/fr/package-catalog/studiocms-plugins/studiocms-wysiwyg.mdx
[uncategorized] ~14-~14: Le point-virgule est précédé d’une espace fine insécable.
Context: ...em } from '@astrojs/starlight/components'; # Présentation Ce plugin ajoute facil...
(FRENCH_WHITESPACE)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: Run Starlight Link Check
🔇 Additional comments (3)
src/content/docs/fr/package-catalog/studiocms-plugins/studiocms-wysiwyg.mdx (3)
1-11: Frontmatter looks solid
La configuration YAML est complète et correctement formatée pour l’i18n.
16-19: Introduction claire et pertinente
Le titre et le paragraphe d’introduction retranscrivent fidèlement le sens du document original.
28-37: Vérifier la syntaxe du bloc de code twoslash
La directive{2, 6}inclut une espace après la virgule alors que la convention courante est{2,6}, et certains moteurs MDX/twoslash pourraient ne pas la reconnaître. Pouvez‑vous confirmer que le format suivant est pris en charge ?```ts twoslash title="studiocms.config.mjs" {2,6}</details> </blockquote></details> </details> <!-- This is an auto-generated comment by CodeRabbit for review status -->
| import { PackageManagers } from 'starlight-package-managers' | ||
| import { Steps, Tabs, TabItem } from '@astrojs/starlight/components'; | ||
|
|
There was a problem hiding this comment.
🛠️ Refactor suggestion
Unify imports & remove non-breaking space
L’import Steps, Tabs, TabItem n’est pas utilisé et contient manifestement une espace fine insécable avant le ;.
- Si vous n’avez pas besoin de ces composants, supprimez la ligne entière.
- Sinon, remplacez la fine insécable par une espace normale ou supprimez tout simplement l’espace avant le
;. - Profitez‑en pour uniformiser la présence de
;dans vos imports (ajouter un;à la ligne 13 ou retirer celui de la ligne 14).
🧰 Tools
🪛 LanguageTool
[uncategorized] ~14-~14: Le point-virgule est précédé d’une espace fine insécable.
Context: ...em } from '@astrojs/starlight/components'; # Présentation Ce plugin ajoute facil...
(FRENCH_WHITESPACE)
There was a problem hiding this comment.
Not used and not uniform, it's true (as in the English version) but you're hallucinating for the non-breaking space! 👀
There was a problem hiding this comment.
@ArmandPhilippot You're absolutely right! I apologize for my error about the non-breaking space. Looking at the code again, there's no evidence of any unusual spacing - the LanguageTool detection gave a false positive that I incorrectly repeated.
Thank you for confirming the unused imports and the uniformity issue, and for catching my mistake!
(￣ ￣\) embarrassed rabbit
dreyfus92
left a comment
dreyfus92
left a comment
There was a problem hiding this comment.
Thank you @ArmandPhilippot 🫡
…s/studiocms-wysiwyg.mdx
ArmandPhilippot
deleted the
i18n/fr-add-package-catalog/studiocms-plugins/studiocms-wysiwyg.mdx
branch
3 participants
Description
Translates in French
package-catalog/studiocms-plugins/studiocms-wysiwyg.mdxadded in #73 and updated in #106Summary by CodeRabbit