Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(i18n): improve french translations - #42470

Merged
rusackas merged 3 commits into
apache:masterfrom
greggailly:gailly/fr_trans
Jul 28, 2026
Merged

fix(i18n): improve french translations#42470
rusackas merged 3 commits into
apache:masterfrom
greggailly:gailly/fr_trans

Conversation

@greggailly

Copy link
Copy Markdown
Contributor

fix(i18n): improve french translations

SUMMARY

Remove many #, fuzzy tags on verified translations.
Updated many translations (some were quite incoherent).
Deleted some duplicates.

TESTING INSTRUCTIONS

Initial pass done by Claude. Each translation has been manually checked and sometimes reverted if given the context the old one was more appropriate.
Native french speaker here but depending on context some translations could have two meanings so happy to dig some of them if needed.

@dosubot dosubot Bot added the i18n:french Translation related to French language label Jul 27, 2026
@github-actions github-actions Bot added the i18n Namespace | Anything related to localization label Jul 27, 2026
@netlify

netlify Bot commented Jul 27, 2026

Copy link
Copy Markdown

Deploy Preview for superset-docs-preview ready!

Name Link
🔨 Latest commit ad184be
🔍 Latest deploy log https://app.netlify.com/projects/superset-docs-preview/deploys/6a67736cc71de300086ee645
😎 Deploy Preview https://deploy-preview-42470--superset-docs-preview.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.

Comment thread superset/translations/fr/LC_MESSAGES/messages.po Outdated
Comment thread superset/translations/fr/LC_MESSAGES/messages.po Outdated
Comment thread superset/translations/fr/LC_MESSAGES/messages.po
@codecov

codecov Bot commented Jul 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.25%. Comparing base (dece457) to head (0d87a56).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #42470   +/-   ##
=======================================
  Coverage   65.25%   65.25%           
=======================================
  Files        2795     2795           
  Lines      157639   157639           
  Branches    36052    36052           
=======================================
  Hits       102860   102860           
  Misses      52802    52802           
  Partials     1977     1977           
Flag Coverage Δ
hive 38.38% <ø> (ø)
mysql 57.55% <ø> (ø)
postgres 57.59% <ø> (ø)
presto 40.30% <ø> (ø)
python 58.99% <ø> (ø)
sqlite 57.21% <ø> (ø)
unit 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bito-code-review

bito-code-review Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Bito Automatic Review Skipped - Large PR

Bito didn't auto-review this change because the pull request exceeded the line limit. No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.

Comment thread superset/translations/fr/LC_MESSAGES/messages.po Outdated
Comment thread superset/translations/fr/LC_MESSAGES/messages.po Outdated
Comment thread superset/translations/fr/LC_MESSAGES/messages.po Outdated

@rusackas rusackas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for going through these by hand instead of just trusting the fuzzy pass. The CodeAnt catches (Saturday/Sunday week-ending flip, the theme-fetch string bleeding into the CSS-template one, labels vs balises) all read correctly on HEAD now, so feel free to resolve those threads.

Activer les étiquettes for "Enable labels" checks out. LGTM!

@rusackas
rusackas merged commit fc59d9c into apache:master Jul 28, 2026
59 checks passed
@greggailly
greggailly deleted the gailly/fr_trans branch July 29, 2026 14:18
rusackas added a commit to ytihianine/superset_fork that referenced this pull request Aug 8, 2026
Semantic three-way merge (by msgid) of the June translation work
against the substantial French catalog activity that landed on
master since: apache#42470, apache#42572, apache#41752 (adopting jeanpommier's 209
fixes), apache#41655/apache#42099 (AI backfills), apache#42577 (85 remaining
untranslated strings), and #1627fab7 (catalog-wide de-fuzzing).

Confirmed human translations are applied and de-fuzzed. Where both
this branch and master independently touched the same entry, kept
whichever reads correctly against the English source and the
catalog's own established terminology (counted, not guessed, e.g.
'étiquette' 74x vs 'balise' 20x, 'mesure' 104x vs 'métrique' 20x).
Rejected edits that introduced anglicisms ('mail' for 'courriel'),
dropped this catalog's inclusive-language convention
('utilisateur·rice'), or otherwise regressed grammar or meaning
relative to the base. Applied genuine fixes to pre-existing master
bugs found along the way (a swapped min/max metric translation,
several stranded/mismatched entries). Restored guillemet typography
(« ») that an editing pass had flattened to straight quotes across
~90 entries, while keeping the underlying translation improvements.

Co-Authored-By: Claude Sonnet 5 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

i18n:french Translation related to French language i18n Namespace | Anything related to localization size/XXL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants