Skip to content

Update textlint with preset-vuejs-jp #375

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

Merged
merged 13 commits into from
Jun 26, 2021

Conversation

naokie
Copy link
Member

@naokie naokie commented Jun 5, 2021

Description of Problem

背景は vuejs-jp/textlint-rule-preset-vuejs-jp#1 を参照してください。

Proposed Solution

vuejs-jp/textlint-rule-preset-vuejs-jp の初導入です。
バージョンの更新やルールセットの追加で検出されたものの fix を含む予定です。

Additional Information

@pr-triage pr-triage bot added the PR: draft label Jun 5, 2021
@naokie
Copy link
Member Author

naokie commented Jun 6, 2021

半角スペースの削除忘れと、コードブロック前後の追加忘れをかなり多く検出できたので、よさそうです。
「1つ2つ」といった単語は頻出するので、想定通りに差分が数多く検出されました。
ここまでは「--fix」で修正が簡単なのでよしとして…

「である/ですます」のチェックがメジャーバージョンアップのためか、文中で何箇所も検出されるようになったので手直しします。

@naokie
Copy link
Member Author

naokie commented Jun 6, 2021

いくつかショックなことがありました。

現在の構造では .textlintrc ファイルが src ディレクトリ配下にあるため、設定が反映されないまま動作していました。
ルート直下、または --config オプションを指定して、動作させると今度は jtf-style プリセットが見つからないというエラーに。
現状は vuejs-jp/textlint-checker-for-vuejs-jp-docs のみルールに含んだチェックをしている状況です。

今回の PR で textlint はルール通りに動作するようになります。
検出される量が増えたのはこのためでした。

@naokie
Copy link
Member Author

naokie commented Jun 13, 2021

3回目のトライ

実行結果
yarn run v1.22.10
$ textlint --format stylish src/**

/h/n/s/g/n/ja.vuejs.org/src/api/application-api.md
   11:146  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であった"
Total:
である  : 2
ですます: 12
            vuejs-jp/no-mix-dearu-desumasu
  126:19   error  箇条書き: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 3
ですます: 28
            vuejs-jp/no-mix-dearu-desumasu
  158:23   error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 2
ですます: 12
            vuejs-jp/no-mix-dearu-desumasu
  228:47   error  箇条書き: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 3
ですます: 28
            vuejs-jp/no-mix-dearu-desumasu
  228:72   error  箇条書き: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であると考えること"
Total:
である  : 3
ですます: 28
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/api/composition-api.md
  109:25  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  110:20  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  111:20  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  112:16  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  113:21  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  114:16  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  115:22  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  116:18  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  117:22  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  118:22  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  119:24  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>

/h/n/s/g/n/ja.vuejs.org/src/community/join.md
  44:125  error  箇条書き: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 1
ですます: 15
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/cookbook/index.md
  24:135  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 4
ですます: 18
  vuejs-jp/no-mix-dearu-desumasu
  60:17   error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 4
ですます: 18
  vuejs-jp/no-mix-dearu-desumasu
  68:181  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 4
ですます: 18
  vuejs-jp/no-mix-dearu-desumasu
  72:109  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 4
ですます: 18
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/a11y-resources.md
  38:80  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 1
ですます: 4
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/class-and-style.md
  3:68  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であるため"
Total:
である  : 1
ですます: 42
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/component-edge-cases.md
  35:67  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である一方"
Total:
である  : 1
ですます: 10
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/component-props.md
   37:26  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である `:`"
Total:
である  : 2
ですます: 29
        vuejs-jp/no-mix-dearu-desumasu
  228:26  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であってもよく、"
Total:
である  : 2
ですます: 29
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/component-registration.md
  156:43  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であれば、"
Total:
である  : 1
ですます: 25
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/composition-api-introduction.md
   11:156  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であること"
Total:
である  : 4
ですます: 58
      vuejs-jp/no-mix-dearu-desumasu
   82:83   error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であるべきです"
Total:
である  : 4
ですます: 58
  vuejs-jp/no-mix-dearu-desumasu
  346:56   error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である `"
Total:
である  : 4
ですます: 58
        vuejs-jp/no-mix-dearu-desumasu
  546:40   error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であること"
Total:
である  : 4
ですます: 58
      vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/composition-api-setup.md
  51:20  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である場合"
Total:
である  : 1
ですます: 20
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/contributing/writing-guide.md
  68:181  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  68:187  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  68:195  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  68:204  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  68:217  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>

/h/n/s/g/n/ja.vuejs.org/src/guide/forms.md
  5:174  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 1
ですます: 25
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/instance.md
  106:230  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である現在"
Total:
である  : 1
ですます: 35
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/introduction.md
   26:68  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であれば、"
Total:
である  : 3
ですます: 45
  vuejs-jp/no-mix-dearu-desumasu
   99:74  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であること"
Total:
である  : 3
ですます: 45
  vuejs-jp/no-mix-dearu-desumasu
  316:78  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である **"
Total:
である  : 3
ですます: 45
   vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/array-refs.md
  70:17  error  箇条書き: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である必要"
Total:
である  : 1
ですます: 1
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/attribute-coercion.md
   9:20  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 2
ですます: 11
      vuejs-jp/no-mix-dearu-desumasu
  55:27  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であれば削除"
Total:
である  : 2
ですます: 11
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/custom-directives.md
  50:11  error  >の対となる<が見つかりません。<>                                              vuejs-jp/4.3.7.山かっこ<>
  85:99  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であるべきこと"
Total:
である  : 1
ですます: 12
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/data-option.md
   43:60  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であるという事実"
Total:
である  : 2
ですます: 4
  vuejs-jp/no-mix-dearu-desumasu
  121:91  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 2
ですます: 4
          vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/filters.md
  41:56  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 1
ですます: 7
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/global-api-treeshaking.md
   37:72   error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である `$"
Total:
である  : 4
ですます: 27
  vuejs-jp/no-mix-dearu-desumasu
   39:85   error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であれ、"
Total:
である  : 4
ですます: 27
   vuejs-jp/no-mix-dearu-desumasu
  110:128  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であれ、"
Total:
である  : 4
ですます: 27
   vuejs-jp/no-mix-dearu-desumasu
  158:49   error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であった"
Total:
である  : 4
ですます: 27
   vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/global-api.md
   60:34  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である _"
Total:
である  : 2
ですます: 21
                  vuejs-jp/no-mix-dearu-desumasu
  112:20  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であるかどうかのチェック"
Total:
である  : 2
ですます: 21
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/introduction.md
  156:96  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 2
ですます: 22
  vuejs-jp/no-mix-dearu-desumasu
  166:50  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である ["
Total:
である  : 2
ですます: 22
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/key-attribute.md
  16:6  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である `"
Total:
である  : 1
ですます: 9
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/listeners-removed.md
  40:72  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であるため"
Total:
である  : 1
ですます: 4
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/suspense.md
   9:18  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 3
ですます: 31
    vuejs-jp/no-mix-dearu-desumasu
  45:71  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であれば、"
Total:
である  : 3
ですます: 31
  vuejs-jp/no-mix-dearu-desumasu
  47:39  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である必要"
Total:
である  : 3
ですます: 31
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/transition.md
  31:26  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 2
ですます: 4
            vuejs-jp/no-mix-dearu-desumasu
  49:15  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であるかがより明確"
Total:
である  : 2
ですます: 4
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/v-model.md
  13:21  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  14:20  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>

/h/n/s/g/n/ja.vuejs.org/src/guide/optimizations.md
  17:78  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である仮想"
Total:
である  : 1
ですます: 12
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/reactivity-computed-watchers.md
   82:110  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であること"
Total:
である  : 2
ですます: 39
  vuejs-jp/no-mix-dearu-desumasu
  142:70   error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であって、"
Total:
である  : 2
ですます: 39
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/reactivity-fundamentals.md
  16:140  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 1
ですます: 26
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/reactivity.md
  107:224  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である**"
Total:
である  : 1
ですます: 76
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/render-function.md
   84:220  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であれば、"
Total:
である  : 2
ですます: 96
  vuejs-jp/no-mix-dearu-desumasu
  298:2    error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であれ、"
Total:
である  : 2
ですます: 96
    vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/ssr/hydration.md
  19:54  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であるか確認"
Total:
である  : 1
ですます: 14
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/ssr/universal.md
  9:16  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であること"
Total:
である  : 2
ですます: 22
        vuejs-jp/no-mix-dearu-desumasu
  9:97  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であるということ"
Total:
である  : 2
ですます: 22
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/state-management.md
   9:25  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であれば、"
Total:
である  : 2
ですます: 16
  vuejs-jp/no-mix-dearu-desumasu
  13:93  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であること"
Total:
である  : 2
ですます: 16
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/template-syntax.md
   60:96  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である場合"
Total:
である  : 2
ですます: 56
        vuejs-jp/no-mix-dearu-desumasu
  194:63  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり後ほど説明"
Total:
である  : 2
ですます: 56
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/testing.md
   69:54   error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であること"
Total:
である  : 2
ですます: 58
      vuejs-jp/no-mix-dearu-desumasu
  100:141  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であるユーザー"
Total:
である  : 2
ですます: 58
  vuejs-jp/no-mix-dearu-desumasu
  113:43   error  <の対となる>が見つかりません。<>                                              vuejs-jp/4.3.7.山かっこ<>

/h/n/s/g/n/ja.vuejs.org/src/guide/transitions-enterleave.md
  503:88  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 1
ですます: 58
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/transitions-list.md
  340:66  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であるため"
Total:
である  : 1
ですます: 31
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/transitions-overview.md
  135:102  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であること"
Total:
である  : 3
ですます: 41
  vuejs-jp/no-mix-dearu-desumasu
  149:80   error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であり、"
Total:
である  : 3
ですます: 41
    vuejs-jp/no-mix-dearu-desumasu
  149:243  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である必要"
Total:
である  : 3
ですます: 41
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/guide/typescript-support.md
   25:80   error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である `"
Total:
である  : 4
ですます: 41
        vuejs-jp/no-mix-dearu-desumasu
  119:56   error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である `"
Total:
である  : 4
ですます: 41
        vuejs-jp/no-mix-dearu-desumasu
  194:37   error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であること"
Total:
である  : 4
ですます: 41
      vuejs-jp/no-mix-dearu-desumasu
  195:112  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であってもです"
Total:
である  : 4
ですます: 41
  vuejs-jp/no-mix-dearu-desumasu

/h/n/s/g/n/ja.vuejs.org/src/style-guide/README.md
  133:80  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であっても、"
Total:
である  : 3
ですます: 116
  vuejs-jp/no-mix-dearu-desumasu
  294:61  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "である可能"
Total:
である  : 3
ですます: 116
    vuejs-jp/no-mix-dearu-desumasu
  389:70  error  本文: "である"調 と "ですます"調 が混在
=> "ですます"調 の文体に、次の "である"調 の箇所があります: "であるため"
Total:
である  : 3
ですます: 116
    vuejs-jp/no-mix-dearu-desumasu

✖ 96 problems (96 errors, 0 warnings)

info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

まだ、96箇所のエラーがありますが、文中に出現する「である」調がほとんどでした。
文体としてそこまで厳密にしなくてもよさそう。
strict モードを OFF にする方針を取るつもりです。

@naokie
Copy link
Member Author

naokie commented Jun 13, 2021

4回目のトライ

Details
yarn run v1.22.10
$ textlint --format stylish src/**

/h/n/s/g/n/ja.vuejs.org/src/api/composition-api.md
  109:25  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  110:20  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  111:20  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  112:16  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  113:21  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  114:16  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  115:22  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  116:18  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  117:22  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  118:22  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  119:24  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>

/h/n/s/g/n/ja.vuejs.org/src/guide/contributing/writing-guide.md
  68:181  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  68:187  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  68:195  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  68:204  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  68:217  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/custom-directives.md
  50:11  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>

/h/n/s/g/n/ja.vuejs.org/src/guide/migration/v-model.md
  13:21  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>
  14:20  error  >の対となる<が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>

/h/n/s/g/n/ja.vuejs.org/src/guide/testing.md
  113:43  error  <の対となる>が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>

✖ 20 problems (20 errors, 0 warnings)

info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
`beforeMount` -> `onBeforeMount`

まだ「->」で山カッコの対がないというルールに該当してしまいます。
これはフィルタで除外をします。

@naokie
Copy link
Member Author

naokie commented Jun 13, 2021

5回目のトライ

Details
yarn run v1.22.10
$ textlint --format stylish src/**

/h/n/s/g/n/ja.vuejs.org/src/guide/testing.md
  113:43  error  <の対となる>が見つかりません。<>  vuejs-jp/4.3.7.山かっこ<>

✖ 1 problem (1 error, 0 warnings)

info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

「< IE11」で山カッコの対がないというルールに該当したのが残りました。
こちらは訳します。

@naokie naokie marked this pull request as ready for review June 13, 2021 03:04
@pr-triage pr-triage bot removed the PR: draft label Jun 13, 2021
@naokie naokie changed the title [WIP] Update textlint with preset-vuejs-jp Update textlint with preset-vuejs-jp Jun 13, 2021
naokie added 4 commits June 13, 2021 12:06
* lang-ja:
  API Reference > Application API の翻訳を追従 (vuejs-jp#371)
  API Reference > Application Config の翻訳を追従 (vuejs-jp#367)
* lang-ja: (28 commits)
  fix: json format quotation
  feature: add Vue 3 vueschool links
  Update custom-directive.md
  Fix: broken gif url
  fix: unify the words and leave colon as it is
  API Reference > Options DOM の翻訳を追従 (vuejs-jp#396)
  API Reference > Options Data の翻訳を追従 (vuejs-jp#395)
  Cookbook の翻訳を追従 (vuejs-jp#394)
  docs: elaborate on how style auto-prefixing works (vuejs-jp#393)
  Update introduction.md (vuejs-jp#392)
  docs: a few improvements (vuejs-jp#391)
  The description of this in setup is inconsistent with the source code (vuejs-jp#388)
  Add onActivated and onDeacitvated hooks to the list (vuejs-jp#387)
  docs: update recommended node version (vuejs-jp#386)
  fix: use method shorthands and switch indexOf to includes (vuejs-jp#385)
  fix: prevent CodePens from stealing focus and scrolling the page (vuejs-jp#384)
  docs: Update key-attribute.md (vuejs-jp#383)
  Add created hook to custom directives code example (vuejs-jp#382)
  API Reference > Global API の翻訳を追従 (vuejs-jp#376)
  guide/typescript-support.mdのtypoを修正(肩→型) (vuejs-jp#403)
  ...
@netlify
Copy link

netlify bot commented Jun 19, 2021

✔️ Deploy Preview for vuejs-v3-ja-doc-preview ready!

🔨 Explore the source changes: dedb883

🔍 Inspect the deploy log: https://app.netlify.com/sites/vuejs-v3-ja-doc-preview/deploys/60d6a35d591c8e0007f42772

😎 Browse the preview: https://deploy-preview-375--vuejs-v3-ja-doc-preview.netlify.app

@naokie
Copy link
Member Author

naokie commented Jun 19, 2021

Netlify の Node バージョンが 12.18.0 であるため、
textlint で必要な最低バージョンに合わずにビルド & デプロイできない状況になりました。

https://github.com/netlify/build-image/blob/xenial/Dockerfile#L268
https://github.com/textlint/textlint/blob/master/packages/textlint/package.json#L100

.nvmrc or .node-version による指定ができるため、v14 系の LTS で乗りきります。

https://docs.netlify.com/configure-builds/manage-dependencies/#node-js-and-javascript

naokie added 2 commits June 26, 2021 12:40
* lang-ja:
  fix: update positioning in lifecycle diagram
  fix: await router push in ssr server
  chore: update partners contact info
@naokie naokie merged commit 3b0485c into vuejs-jp:lang-ja Jun 26, 2021
@naokie naokie deleted the feature/apply-new-textlint-preset branch June 26, 2021 03:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant