Skip to content

Conversation

@hiroshiyui
Copy link
Contributor

Remove the weird phrase-bopomofo mapping entry '舒適性'

舒適性,100,ㄒㄧㄥˋ

And there is a correct entry '舒適性' in the tsi.csv already:

舒適性,100,ㄕㄨ ㄕˋ ㄒㄧㄥˋ

related issue: #49

@kanru
Copy link
Member

kanru commented Aug 18, 2025

I added a lint!

It detects more errors. Would you like to also fix them?

Parsing failed at line 24707: 博物學,2,ㄅㄛˊ ㄨˋ: word count doesn't match
Parsing failed at line 46332: 小港鄉,0,ㄒㄧㄠˇ ㄍㄤˇ: word count doesn't match
Parsing failed at line 75330: 東山區,372,ㄉㄨㄥ ㄕㄢ: word count doesn't match
Parsing failed at line 142673: 酸鹼鹽,14,ㄙㄨㄢ ㄧㄢˊ: word count doesn't match

@hiroshiyui
Copy link
Contributor Author

Cool! Let me fix them.

I added a lint!

It detects more errors. Would you like to also fix them?

Parsing failed at line 24707: 博物學,2,ㄅㄛˊ ㄨˋ: word count doesn't match
Parsing failed at line 46332: 小港鄉,0,ㄒㄧㄠˇ ㄍㄤˇ: word count doesn't match
Parsing failed at line 75330: 東山區,372,ㄉㄨㄥ ㄕㄢ: word count doesn't match
Parsing failed at line 142673: 酸鹼鹽,14,ㄙㄨㄢ ㄧㄢˊ: word count doesn't match

@hiroshiyui hiroshiyui force-pushed the fix-weird-phrase-bopomofo-mapping branch from b5b50dc to f0bfd40 Compare August 18, 2025 12:47
@hiroshiyui
Copy link
Contributor Author

@kanru I have been updated the phrase source file, please review it.

@kanru kanru merged commit 0146546 into chewing:master Aug 18, 2025
1 check passed
@kanru
Copy link
Member

kanru commented Aug 18, 2025

Thanks!

@hiroshiyui hiroshiyui deleted the fix-weird-phrase-bopomofo-mapping branch August 18, 2025 12:59
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.

2 participants