Skip to content

i18n(fr): update configuration-reference.mdx#13826

Open
ArmandPhilippot wants to merge 11 commits into
withastro:mainfrom
ArmandPhilippot:i18n/fr-update-configuration-reference
Open

i18n(fr): update configuration-reference.mdx#13826
ArmandPhilippot wants to merge 11 commits into
withastro:mainfrom
ArmandPhilippot:i18n/fr-update-configuration-reference

Conversation

@ArmandPhilippot
Copy link
Copy Markdown
Member

@ArmandPhilippot ArmandPhilippot commented May 4, 2026

Description (required)

Adds changes from #13802, #13784, #13806, #13844, and #13849 to the French translations of configuration-reference.mdx and integrations-reference.mdx.

Related issues & labels (optional)

  • Suggested label: i18n

@netlify
Copy link
Copy Markdown

netlify Bot commented May 4, 2026

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 17c2d9b
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/6a0845a01335d90008056b33
😎 Deploy Preview https://deploy-preview-13826--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label May 4, 2026
@astrobot-houston
Copy link
Copy Markdown
Contributor

astrobot-houston commented May 4, 2026

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
fr/reference/configuration-reference.mdx Localization changed, will be marked as complete. 🔄️
fr/reference/integrations-reference.mdx Localization changed, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@ArmandPhilippot ArmandPhilippot marked this pull request as draft May 5, 2026 06:27
@ArmandPhilippot ArmandPhilippot marked this pull request as ready for review May 5, 2026 06:38
Comment thread src/content/docs/fr/reference/configuration-reference.mdx Outdated
Co-authored-by: Yan <61414485+yanthomasdev@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@trueberryless trueberryless left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your translation!

LGTM 🎉

Further potential improvements (outside diff range)

  • src/content/docs/fr/reference/integrations-reference.mdx line 1424: Heading structure mismatch: the original English file has 14 h3 subsection(s) under this h2 section, but this translation has 15. Please check for missing or extra headings.

These are out of scope for this PR, so let's create follow-up PR(s) for them 👍


EDIT: Hi Armand! I'm testing a little project I am working on and it discovered that the French translation of integrations-reference.mdx still includes the Importations de types dépréciées section, which got removed in the English doc I think. That's what the message above is about...

@ArmandPhilippot
Copy link
Copy Markdown
Member Author

ArmandPhilippot commented May 16, 2026

Ah, nice catch! It seems I forgot a change in #12322 ... I double checked; the line numbers now match. Thanks!

Edit: but the line number was wrong, the section was at line 2623 not at line 1424.

@trueberryless
Copy link
Copy Markdown
Contributor

trueberryless commented May 16, 2026

Ah, nice catch! It seems I forgot a change in #12322 ... I double checked; the line numbers now match. Thanks!

Edit: but the line number was wrong, the section was at line 2623 not at line 1424.

Thank you very much for the confirmation!
I only noticed because I am currently building a little tool to catch common "mistakes" for Astro docs: https://astro-docs-i18n-reviewer.trueberryless.org/

But it's still in the early phase, so thanks for the feedback with the line number 👍

Edit: Looking closer at this specific case, depending on how you would determine the suggestion line, line 1424 is technically correct for saying: "this is the heading for the section which has a heading mismatch under it" (so it's the first possible line where the heading mismatch affects the missing / additional part)...

Another option would be to suggest on the last possible heading line of the mismatching section, I'm actually not sure what would be nearer in most cases... I'll have to think about it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants