-
Notifications
You must be signed in to change notification settings - Fork 80
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
fix(deinflect): Only deinflect って and friends to く for 行く #1974
The head ref may contain hidden characters: "fix-\u304F-inflections"
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left two comments which should answer the questions you had in the PR description. Sorry if my previous explanations were unclear. (Also sorry that this part of the code base hasn't been cleaned up to be easier to edit)
No problem, thanks for explaining and helping me. |
Why did the tests failed this time? 🤔 |
Not sure; for now I'm just re-running them in case it was a fluke. |
Looks like that worked. I will lightly edit the PR title/description for style and then approve which will start the merge process. (The PR title and body become the commit message so they're a bit more important than in other repos) |
@all-contributors please add @bruno-queiroz for code. |
I've put up a pull request to add @bruno-queiroz! 🎉 |
Adds @bruno-queiroz as a contributor for code. This was requested by melink14 [in this comment](#1974 (comment))
Thank you 😁 |
🎉 This PR is included in version 2.5.45 🎉 The release is available on: Your semantic-release bot 📦🚀 |
We don't match the kana form in order to not incorrectly match いって to 往く, which would be incorrect. 行く is almost always written with kanji so it seems like the right tradeoff.
Fixes #1920