Skip to content

Normalize 「薩列里」 spelling across phrase dictionaries#1184

Merged
frankslin merged 2 commits into
masterfrom
copilot/modify-salary-term
May 8, 2026
Merged

Normalize 「薩列里」 spelling across phrase dictionaries#1184
frankslin merged 2 commits into
masterfrom
copilot/modify-salary-term

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 8, 2026

  • 确认 TWPhrases/TWPhrasesRev 中「薩列里 薩列里」位置与对称约束
  • 改动前运行现有词典相关构建/测试基线检查
  • 从 TWPhrases.txt 删除「薩列里 薩列里」
  • 从 TWPhrasesRev.txt 删除「薩列里 薩列里」
  • 改动后运行构建/测试验证
  • 执行并处理并行校验(Code Review + CodeQL)
  • 在 PR 说明中明确:此前 薩列裡 -> 薩列里 为有意字符规范化(裡→里),本次仅删除 TW 双向词典中的同形映射

@danny0838
Copy link
Copy Markdown
Contributor

danny0838 commented May 8, 2026

薩列里 => 薩列里 同形,有需要留在表中嗎?

@frankslin
Copy link
Copy Markdown
Collaborator

frankslin commented May 8, 2026

TWPhrases/TWPhrasesRev 删除;STPhrases 保留,因为作为分词界限还是有意义的。

Copilot AI requested a review from frankslin May 8, 2026 02:08
@frankslin frankslin merged commit 445c6cb into master May 8, 2026
36 of 37 checks passed
@frankslin frankslin deleted the copilot/modify-salary-term branch May 8, 2026 02:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants