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

fix(deps): update to kuromojin@3 #6

Merged
merged 6 commits into from
Apr 24, 2021
Merged

fix(deps): update to kuromojin@3 #6

merged 6 commits into from
Apr 24, 2021

Conversation

bizen241
Copy link
Contributor

Summary

Update to kuromojin@3 and improve analysis.

Fixes

Copy link
Member

@azu azu left a comment

Choose a reason for hiding this comment

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

Thanks!

package.json Outdated
},
"keywords": [
"textlint",
"textlintrule"
],
"description": "ら抜き言葉を検出するtextlint rule",
"devDependencies": {
"husky": "^4.3.0",
"lint-staged": "^10.4.2",
"lint-staged": "^10.5.4",
"power-assert": "^1.4.1",
Copy link
Member

@azu azu Apr 24, 2021

Choose a reason for hiding this comment

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

Probably, we can remove power-assert dependencies.
It looks no longer used.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you for reviewing.
I'll remove it.

Copy link
Member

@azu azu left a comment

Choose a reason for hiding this comment

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

LGTM!

@azu azu merged commit 76a27f2 into textlint-ja:master Apr 24, 2021
@azu
Copy link
Member

azu commented Apr 24, 2021

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