search

azu / kuromojin

Provide a high-level wrapper for kuromoji.js. Cache/Promise API
https://kuromojin.netlify.app/
MIT License
87 stars 9 forks source link

Update to kuromoji.js 0.1.2: HELP WANTED 🙇 🆘 #8

Closed azu closed 3 years ago

azu commented 3 years ago

kuromoji.js 0.1.2 has a breaking change.

We will update kuromoji.js@0.1.1 → 0.12 as major update kuromojin@3.

After that, we need to update textlint rules that depended on kuromoji@2. Some rules are broken and we need to fix it!

If you want to help us, please comments!

アップデートのPRを手伝ってくれる人は、どれをやるとかのコメントお願いします。

Plan

  1. Release kuromoji@3 that use kuromoji.js 0.1.2
  2. Update all dependant rules - migrate kuromoji@2 to kuromoji@3
    • ~If no effect, publish as a minor updates~
    • Publish as major updates
  3. Update all presets

Need to update rules

📝 azu/migrate-travis-ci-to-github-actions: Migrate Travis CI to GitHub Actions. Node.js CI settingsを使ってTravis CI → GitHub Actionsもあわせてやるかも。

壊れたルールの例。

https://gist.github.com/azu/8f4435141a5eefe9dcce41a3652ede0b

azu commented 3 years ago

https://github.com/takuyaa/kuromoji.js/issues/23 0.1.1 and 0.1.2 difference results.

azu commented 3 years ago

We need to fix max-ten before updating.

azu commented 3 years ago

I've released kuromojin@3.0.0. We will start migration!

https://github.com/azu/kuromojin/releases/tag/v3.0.0

This uses the latest kuromoji.js https://takuyaa.github.io/kuromoji.js/demo/tokenize.html

DIFF

0.1.2 https://takuyaa.github.io/kuromoji.js/demo/tokenize.html 0.1.1

https://7e32v.csb.app/

azu commented 3 years ago

I will take https://github.com/textlint-ja/textlint-rule-no-double-negative-ja

PR: https://github.com/textlint-ja/textlint-rule-no-double-negative-ja/pull/5

azu commented 3 years ago

基本的にmajorアップデートで揃える方が楽かもしれない。 そのあとpresetもmajorアップデートする形

azu commented 3 years ago

textlint-ja/analyze-desumasu-dearu: 文の敬体(ですます調)、常体(である調)を解析するJavaScriptライブラリ

I took it. https://github.com/textlint-ja/analyze-desumasu-dearu/pull/17

azu commented 3 years ago

I took it. https://github.com/textlint-ja/textlint-rule-no-mix-dearu-desumasu/pull/28 with https://github.com/textlint-ja/textlint-rule-no-mix-dearu-desumasu/pull/28

azu commented 3 years ago

textlint-ja/textlint-rule-morpheme-match: 形態素解析結果のTokenベースの辞書でマッチするtextlintルール https://github.com/azu/morpheme-match is library

I took it.

azu commented 3 years ago

textlint-ja/textlint-rule-ja-no-abusage: よくある日本語の誤用をチェックするtextlintルール

I took it. https://github.com/textlint-ja/textlint-rule-ja-no-abusage/pull/20

hata6502 commented 3 years ago

hata6502/textlint-rule-ja-no-orthographic-variants: 表記ゆれをチェックするtextlintルール hata6502/textlint-rule-ja-no-inappropriate-words: 不適切表現をチェックするtextlintルール hata6502/textlint-rule-no-hoso-kinshi-yogo: 放送禁止用語をチェックするtextlintルール

やります。

azu commented 3 years ago

@hata6502 ありがとうございます！ もしよろしければ https://github.com/textlint-ja にリポジトリを移すのもいいかもしれません。 必要ならInviteをお送りします。

bizen241 commented 3 years ago

I will take textlint-ja/textlint-rule-no-dropping-the-ra: ら抜き言葉をチェックするtextlintルール

azu commented 3 years ago

I take textlint-ja/textlint-rule-no-doubled-joshi: 文中に同じ助詞が複数出てくるのをチェックするtextlintルール

azu commented 3 years ago

textlint-ja/textlint-rule-no-dropping-i: い抜き言葉をチェックするtextlintルール

I take it.

bizen241 commented 3 years ago

I will take textlint-ja/textlint-rule-ja-no-successive-word: 同一の単語を間違えて連続しているのを見つけるtextlintルール

hata6502 commented 3 years ago

@azu

hata6502/textlint-rule-ja-no-orthographic-variants: 表記ゆれをチェックするtextlintルール hata6502/textlint-rule-ja-no-inappropriate-words: 不適切表現をチェックするtextlintルール hata6502/textlint-rule-no-hoso-kinshi-yogo: 放送禁止用語をチェックするtextlintルール

アップデート完了しました！npm にも publish してあります。 hata6502/textlint-rule-no-hoso-kinshi-yogo については、 dependabot によるセキュリティアップデートも取り込んでいます。

azu commented 3 years ago

@hata6502 ありがとうございます

azu commented 3 years ago

textlint-ja/textlint-rule-ja-no-redundant-expression: 冗長な表現をチェックするtextlintルール

I take it

hata6502 commented 3 years ago

もしよろしければ https://github.com/textlint-ja にリポジトリを移すのもいいかもしれません。

こちらとても興味があります。 現状、当方でメンテナンスしている日本語ルールは以下のとおりです。

クオリティ低めのプラグインを除いて、 ぜひ https://github.com/textlint-ja に移譲したいと存じます。

azu commented 3 years ago

@hata6502 GitHubとnpmにそれぞれ招待を送りました！ https://github.com/textlint-ja/textlint-ja#for-new-member の手順に従ってGitHubリポジトリとnpmのアクセス権をそれぞれorganizationに移譲できます。

azu commented 3 years ago

リポジトリを移した後にpackage.jsonとかREADMEのリンク先を変えるのにtransfer-github-ownerが使えたりします。

$ npx transfer-github-owner --from  hata6502 --to textlint-ja

とかでだいたい書き換えてくれます。(ただの置換なので別に使わなくても問題ないです)

azu commented 3 years ago

textlint-ja/textlint-rule-no-insert-dropping-sa: サ抜き、サ入れ表現の誤用をチェックするtextlintルール

Last one! https://github.com/textlint-ja/textlint-rule-no-insert-dropping-sa/pull/7

azu commented 3 years ago

Complete ✅

@hata6502 @bizen241 ありがとうございました！

azu commented 3 years ago

https://github.com/textlint-ja/textlint-rule-preset-ja-technical-writing/issues/73#issuecomment-826238597

が抜けていたのでそれぞれ更新した