Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat change tone #181

Merged
merged 3 commits into from
Mar 17, 2024
Merged

Feat change tone #181

merged 3 commits into from
Mar 17, 2024

Conversation

zh-lx
Copy link
Owner

@zh-lx zh-lx commented Mar 17, 2024

PR 的功能

请详细描述问题,或者贴一个 issue 链接

你的预期是什么

请详细描述,你修改代码之后的样子

是否进行了详细的自测?

是/否

@zh-lx zh-lx merged commit c4aa24e into main Mar 17, 2024
1 of 2 checks passed
This was linked to issues Mar 17, 2024
@zh-lx zh-lx mentioned this pull request Mar 17, 2024
it('[special change tone]一面', () => {
const result = pinyin('一面');
expect(result).to.be.equal('yí miàn');
});
Copy link
Collaborator

Choose a reason for hiding this comment

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

建议加一下 「一会儿」的测试,当前变调失败:

image

Copy link
Owner Author

Choose a reason for hiding this comment

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

是不是因为我还没发包?

Copy link
Collaborator

Choose a reason for hiding this comment

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

我用的这个分支。

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