bito-code-review[bot] commented on code in PR #41652:
URL: https://github.com/apache/superset/pull/41652#discussion_r3509939433


##########
superset/translations/th/LC_MESSAGES/messages.po:
##########
@@ -30763,11 +30727,8 @@ msgstr "สร้างแผนภูมิใหม่"
 msgid "create dataset from SQL query"
 msgstr "สร้างชุดข้อมูลจากคำสั่ง SQL"
 
-# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: ca, cs, de,
-# es, fa, fr, lv, mi, sk, sl, uk]
-#, fuzzy
 msgid "crontab"
-msgstr "crontab"
+msgstr ""

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>Translation regression: crontab</b></div>
   <div id="fix">
   
   This change replaces an existing translation with an empty string. The 
previous `msgstr "crontab"` was a valid machine-translated entry (confirmed by 
the `# Machine-translated via backfill_po.py (claude-sonnet-4-6)` comment). The 
`_is_missing()` check at `backfill_po.py:152` treats empty `msgstr` as a 
missing translation, so this change would re-queue `crontab` for 
re-translation, undoing prior work.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #07ef2b</i></small>
   </div>
   
   ---
   Should Bito avoid suggestions like this for future reviews? (<a 
href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>)
   - [ ] Yes, avoid them



##########
superset/translations/th/LC_MESSAGES/messages.po:
##########
@@ -31126,11 +31087,8 @@ msgstr "รายการต่อหน้า"
 msgid "error"
 msgstr "ข้อผิดพลาด"
 
-# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: ar, ca, cs,
-# de, es, fa, ja, lv, mi, nl, pt_BR, sk, sl, uk]
-#, fuzzy
 msgid "error_message"
-msgstr "error_message"
+msgstr ""

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>Translation regression: error_message</b></div>
   <div id="fix">
   
   This change replaces an existing translation with an empty string. The 
previous `msgstr "error_message"` was a valid machine-translated entry 
(confirmed by the `# Machine-translated via backfill_po.py (claude-sonnet-4-6)` 
comment). The `_is_missing()` check at `backfill_po.py:152` treats empty 
`msgstr` as a missing translation, so this change would re-queue 
`error_message` for re-translation, undoing prior work.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #07ef2b</i></small>
   </div>
   
   ---
   Should Bito avoid suggestions like this for future reviews? (<a 
href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>)
   - [ ] Yes, avoid them



##########
superset/translations/th/LC_MESSAGES/messages.po:
##########
@@ -31859,11 +31812,8 @@ msgstr ""
 "ชุดข้อมูลทั้งหมดใช้สเกลเดียวกัน; การเปลี่ยนแปลง: "
 "แสดงการเปลี่ยนแปลงเปรียบเทียบกับจุดข้อมูลแรกในแต่ละชุดข้อมูล"
 
-# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: ca, cs, de,
-# es, fa, fr, ja, lv, mi, pl, pt_BR, sk, sl, uk]
-#, fuzzy
 msgid "sql"
-msgstr "sql"
+msgstr ""

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>Translation regression: sql</b></div>
   <div id="fix">
   
   This change replaces an existing translation with an empty string. The 
previous `msgstr "sql"` was a valid machine-translated entry (confirmed by the 
`# Machine-translated via backfill_po.py (claude-sonnet-4-6)` comment). The 
`_is_missing()` check at `backfill_po.py:152` treats empty `msgstr` as a 
missing translation, so this change would re-queue `sql` for re-translation, 
undoing prior work.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #07ef2b</i></small>
   </div>
   
   ---
   Should Bito avoid suggestions like this for future reviews? (<a 
href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>)
   - [ ] Yes, avoid them



##########
superset/translations/th/LC_MESSAGES/messages.po:
##########
@@ -31829,11 +31785,8 @@ msgstr "กำลังทำงาน"
 msgid "save"
 msgstr "บันทึก"
 
-# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: cs, ja, lv,
-# sk, uk]
-#, fuzzy
 msgid "schema1,schema2"
-msgstr "schema1,schema2"
+msgstr ""

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>Translation regression: schema1,schema2</b></div>
   <div id="fix">
   
   This change replaces an existing translation with an empty string. The 
previous `msgstr "schema1,schema2"` was a valid machine-translated entry 
(confirmed by the `# Machine-translated via backfill_po.py (claude-sonnet-4-6)` 
comment). The `_is_missing()` check at `backfill_po.py:152` treats empty 
`msgstr` as a missing translation, so this change would re-queue 
`schema1,schema2` for re-translation, undoing prior work.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #07ef2b</i></small>
   </div>
   
   ---
   Should Bito avoid suggestions like this for future reviews? (<a 
href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>)
   - [ ] Yes, avoid them



##########
superset/translations/th/LC_MESSAGES/messages.po:
##########
@@ -31707,10 +31665,8 @@ msgstr ""
 msgid "permalink state not found"
 msgstr "ไม่พบสถานะลิงก์ถาวร"
 
-# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: ja, lv, uk]
-#, fuzzy
 msgid "pivoted_xlsx"
-msgstr "pivoted_xlsx"
+msgstr ""

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>Translation regression: pivoted_xlsx</b></div>
   <div id="fix">
   
   This change replaces an existing translation with an empty string. The 
previous `msgstr "pivoted_xlsx"` was a valid machine-translated entry 
(confirmed by the `# Machine-translated via backfill_po.py (claude-sonnet-4-6)` 
comment). The `_is_missing()` check at `backfill_po.py:152` treats empty 
`msgstr` as a missing translation, so this change would re-queue `pivoted_xlsx` 
for re-translation, undoing prior work.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #07ef2b</i></small>
   </div>
   
   ---
   Should Bito avoid suggestions like this for future reviews? (<a 
href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>)
   - [ ] Yes, avoid them



##########
superset/translations/th/LC_MESSAGES/messages.po:
##########
@@ -31889,17 +31839,11 @@ msgstr "สลับกัน"
 msgid "std"
 msgstr "std"
 
-# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: ar, ca, cs,
-# de, es, fa, ja, lv, mi, nl, pt_BR, sk, sl, uk]
-#, fuzzy
 msgid "step-after"
-msgstr "ขั้น-หลัง"
+msgstr ""

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>Translation regression: step-after</b></div>
   <div id="fix">
   
   This change replaces an existing translation with an empty string. The 
previous `msgstr "ขั้นหลัง"` was a valid machine-translated entry for 
step-after (confirmed by the `# Machine-translated via backfill_po.py 
(claude-sonnet-4-6)` comment on line 31898). The `_is_missing()` check at 
`backfill_po.py:152` treats empty `msgstr` as a missing translation, so this 
change would re-queue step-after for re-translation, undoing prior work.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #07ef2b</i></small>
   </div>
   
   ---
   Should Bito avoid suggestions like this for future reviews? (<a 
href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>)
   - [ ] Yes, avoid them



##########
superset/translations/th/LC_MESSAGES/messages.po:
##########
@@ -31889,17 +31839,11 @@ msgstr "สลับกัน"
 msgid "std"
 msgstr "std"
 
-# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: ar, ca, cs,
-# de, es, fa, ja, lv, mi, nl, pt_BR, sk, sl, uk]
-#, fuzzy
 msgid "step-after"
-msgstr "ขั้น-หลัง"
+msgstr ""
 
-# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: ar, ca, cs,
-# de, es, fa, fr, ja, lv, mi, nl, pl, pt_BR, sk, sl, uk, zh, zh_TW]
-#, fuzzy
 msgid "step-before"
-msgstr "ขั้น-ก่อน"
+msgstr ""

Review Comment:
   <div>
   
   
   <div id="suggestion">
   <div id="issue"><b>Translation regression: step-before</b></div>
   <div id="fix">
   
   This change replaces an existing translation with an empty string. The 
previous `msgstr "ขั้นก่อน"` was a valid machine-translated entry for 
step-before (confirmed by the `# Machine-translated via backfill_po.py 
(claude-sonnet-4-6)` comment on line 31900). The `_is_missing()` check at 
`backfill_po.py:152` treats empty `msgstr` as a missing translation, so this 
change would re-queue step-before for re-translation, undoing prior work.
   </div>
   
   
   </div>
   
   
   
   
   <small><i>Code Review Run #07ef2b</i></small>
   </div>
   
   ---
   Should Bito avoid suggestions like this for future reviews? (<a 
href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>)
   - [ ] Yes, avoid them



-- 
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]

Reply via email to