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

閉じかっこの直前にある連続した句読点の扱いについて #46

Open
massongit opened this issue Sep 13, 2020 · 1 comment

Comments

@massongit
Copy link

massongit commented Sep 13, 2020

masterブランチでテストを実行すると、以下のテストケースで失敗します。

{
text: "「こんにちは、世界。。。」",
output: "「こんにちは、世界」",
errors: [
{
message: "句読点(。、)が閉じ括弧の直前に存在しています",
line: 1,
column: 12
},
{
message: "連続した句読点(。、)が使われています",
line: 1,
column: 10
}
]
},

$ yarn test
...
  1) general-novel-style Fixer: 「こんにちは、世界。。。」:

      AssertionError [ERR_ASSERTION] [ERR_ASSERTION]: Expected values to be strictly equal:
+ actual - expected

+ '「こんにちは、世界……」'
- '「こんにちは、世界」'
            ^
      + expected - actual

      -「こんにちは、世界……」
      +「こんにちは、世界」
      
      at node_modules/textlint-tester/src/textlint-tester.js:72:28

こちらのテストケースでは2つの指摘がなされますが、修正時の優先順位としては 句読点(。、)が閉じ括弧の直前に存在しています (句読点削除) > 連続した句読点(。、)が使われています (三点リーダーに修正) となるのが仕様上正しいのでしょうか?

@massongit
Copy link
Author

テストが正として修正のPRを出しました: #47

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

No branches or pull requests

1 participant