rusackas commented on PR #38948:
URL: https://github.com/apache/superset/pull/38948#issuecomment-5227685184
Thanks again for sticking with this one, @FrancescoCastaldi. Rebased and
reconciled against current `master`, which has had five separate
Italian-catalog commits land since this branch's base, so the raw diff came out
to 2227 conflicting entries. That's too many to adjudicate one by one, so
here's the systematic approach and what fell out of it, in case any of it's
worth a second look on your end.
**Still broken as submitted, excluded (kept `master`'s version):**
- 97 entries with mismatched `%()s`/`{}` placeholders relative to their
`msgid` — the same class of bug from my last pass, still present.
- ~180 more from a related defect: long help-text translations truncated
mid-sentence (looks like an AI-generation cutoff), a batch of
placeholder-example strings (`e.g. 127.0.0.1`, `e.g. AccountAdmin`, `e.g.
5432`, ten total) that had all collapsed to the identical generic "per
esempio.", and about 20 short technical terms/proper nouns translated as false
friends — `STRING` → `CORDA` (rope), `deckGL` → `mazzoGL` (deck of cards),
`sql` → `mq` (square meters), `mean` → `Significare` (to signify), `Time Grain`
→ `Grano` (wheat), `Sparkline` → literally "sparkling line". Found these by
reading every single 1-2 word changed entry in full, not sampling, since short
entries are where false-friend errors hide best.
**~1265 entries where both sides have a confirmed, differing translation** —
kept `master`'s by default. Not a quality judgment on your phrasing
specifically, just that adjudicating over a thousand individual disagreements
isn't reviewable at this scale, and `master`'s confirmed content passed a full
structural check clean while this branch didn't.
**Applied ~783 entries** — either filling a gap where `master` was still
fuzzy/AI-backfilled, or uncontested improvements, all passing the placeholder
and completeness checks above.
Catalog compiles clean under `msgfmt -c`, zero new fuzzy entries. Happy to
hand over the exclusion lists if useful for a follow-up pass on the ~1265
disagreements, that's genuinely the highest-value remaining work here.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
