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

refactor(textlint): textlint-rule-ja-no-redundant-expressionの検知を元に修正 #614

Merged
merged 25 commits into from
Jan 12, 2019

Conversation

azu
Copy link
Collaborator

@azu azu commented Dec 26, 2018

@azu
Copy link
Collaborator Author

azu commented Dec 26, 2018

まだLintエラーが残ってるけど、次のケースはそのままでも違和感がないかなと思った箇所。

大体意味的には、"ソフトウェアを実行"の趣旨だと思う。

preset-ja-technical-writing/ja-no-redundant-expression: "プログラムを実行"は冗長な表現です。"プログラムする"など簡潔な表現にすると文章が明瞭になります。参考: http://www.atmarkit.co.jp/ait/articles/1001/19/news106_2.html
/Users/azu/.ghq/github.com/asciidwango/js-primer/source/basic/read-eval-print/README.md:274:8
                        v
    273. 
    274. 実行時エラーはプログラムを実行している最中に発生するエラーです。
    275. 実行時(ランタイム)におきるエラーであるためランタイムエラーと呼ばれることもあります。
                        ^

preset-ja-technical-writing/ja-no-redundant-expression: "処理を実行"は冗長な表現です。"処理する"など簡潔な表現にすると文章が明瞭になります。参考: http://www.atmarkit.co.jp/ait/articles/1001/19/news106_2.html
/Users/azu/.ghq/github.com/asciidwango/js-primer/source/basic/function-declaration/README.md:8:45
                                                                                                v
    7. 関数とは、ある一連の手続き(文の集まり)を1つの処理としてまとめる機能です。
    8. 関数を利用することで、同じ処理を毎回書くのではなく、一度定義した関数を呼び出すことで同じ処理を実行できます。
    9. 
                                                                                                ^

preset-ja-technical-writing/ja-no-redundant-expression: "処理を実行"は冗長な表現です。"処理する"など簡潔な表現にすると文章が明瞭になります。参考: http://www.atmarkit.co.jp/ait/articles/1001/19/news106_2.html
/Users/azu/.ghq/github.com/asciidwango/js-primer/source/basic/statement-expression/README.md:41:26
                                              v
    40. 
    41. **文**(Statement)を簡潔に述べると、処理を実行する1ステップが1つの文といえます。
    42. JavaScriptでは、文の末尾にセミコロン(`;`)を置くことで文と文に区切りを付けます。
                                              ^

preset-ja-technical-writing/ja-no-redundant-expression: "処理を実行"は冗長な表現です。"処理する"など簡潔な表現にすると文章が明瞭になります。参考: http://www.atmarkit.co.jp/ait/articles/1001/19/news106_2.html
/Users/azu/.ghq/github.com/asciidwango/js-primer/source/basic/async/README.md:121:42
                                                                                v
    120. 次のコードは、`setTimeout`関数でタイマーに登録したコールバック関数が呼ばれるまで、実際にどの程度の時間がかかったかを計測しています。
    121. また、`setTimeout`関数でタイマーに登録した次の行で同期的にブロックする処理を実行しています。
    122. 
                                                                                ^

preset-ja-technical-writing/ja-no-redundant-expression: "処理を実行"は冗長な表現です。"処理する"など簡潔な表現にすると文章が明瞭になります。参考: http://www.atmarkit.co.jp/ait/articles/1001/19/news106_2.html
/Users/azu/.ghq/github.com/asciidwango/js-primer/source/basic/async/README.md:1130:75
                                                                                                                            v
    1129. `Promise.all`メソッドは複数のPromiseがすべて完了するまで待つ処理でした。
    1130. `Promise.race`メソッドでは複数のPromiseを受け取りますが、Promiseが1つでも完了した(Settle状態となった)時点で次の処理を実行します。
    1131. 
                                                                                                                            ^

preset-ja-technical-writing/ja-no-redundant-expression: "処理を行う"は冗長な表現です。"処理する"など簡潔な表現にすると文章が明瞭になります。参考: http://www.atmarkit.co.jp/ait/articles/1001/19/news106_2.html
/Users/azu/.ghq/github.com/asciidwango/js-primer/source/basic/async/README.md:1228:20
                                   v
    1227. 
    1228. Async Functionとは非同期処理を行う関数を定義する構文です。
    1229. Async Functionは通常の関数とは異なり、必ず`Promise`インスタンスを返す関数を定義します。
                                   ^

preset-ja-technical-writing/ja-no-redundant-expression: "処理を実行"は冗長な表現です。"処理する"など簡潔な表現にすると文章が明瞭になります。参考: http://www.atmarkit.co.jp/ait/articles/1001/19/news106_2.html
/Users/azu/.ghq/github.com/asciidwango/js-primer/source/basic/async/README.md:1361:15
                                       v
    1360. 
    1361. 普通の処理の流れでは、非同期処理を実行した場合にその非同期処理の完了を待つことなく、次の行(次の文)を実行します。
    1362. しかし`await`式では非同期処理を実行し完了するまで、次の行(次の文)を実行しません。
                                       ^

preset-ja-technical-writing/ja-no-redundant-expression: "処理を実行"は冗長な表現です。"処理する"など簡潔な表現にすると文章が明瞭になります。参考: http://www.atmarkit.co.jp/ait/articles/1001/19/news106_2.html
/Users/azu/.ghq/github.com/asciidwango/js-primer/source/basic/async/README.md:1362:17
                                    v
    1361. 普通の処理の流れでは、非同期処理を実行した場合にその非同期処理の完了を待つことなく、次の行(次の文)を実行します。
    1362. しかし`await`式では非同期処理を実行し完了するまで、次の行(次の文)を実行しません。
    1363. そのため`await`式を使うことで非同期処理が同期処理のように上から下へと順番に実行するような処理順で書けます。
                                    ^

preset-ja-technical-writing/ja-no-redundant-expression: "処理を行う"は冗長な表現です。"処理する"など簡潔な表現にすると文章が明瞭になります。参考: http://www.atmarkit.co.jp/ait/articles/1001/19/news106_2.html
/Users/azu/.ghq/github.com/asciidwango/js-primer/source/use-case/ajaxapp/promise/README.md:85:27
                                                   v
    84. 
    85. Promiseのコンストラクタに渡す関数で、XHRの処理を行います。
    86. 作成されたPromiseは成功か失敗のどちらかで完了させなければなりません。
                                                   ^

preset-ja-technical-writing/ja-no-redundant-expression: "テストを行う"は冗長な表現です。"テストする"など簡潔な表現にすると文章が明瞭になります。参考: http://www.atmarkit.co.jp/ait/articles/1001/19/news106_2.html
/Users/azu/.ghq/github.com/asciidwango/js-primer/source/use-case/nodecli/refactor-and-unittest/README.md:13:5
                 v
    12. 前のセクションまでは、すべての処理をひとつのJavaScriptファイルに記述していました。
    13. ユニットテストを行うためにはテスト対象がモジュールとして分割されていなければいけません。
    14. 今回のアプリケーションでは、CLIアプリケーションとしてコマンドライン引数を処理する部分と、MarkdownをHTMLへ変換する部分に分割します。
                 ^

preset-ja-technical-writing/ja-no-redundant-expression: "テストを実行"は冗長な表現です。"テストする"など簡潔な表現にすると文章が明瞭になります。参考: http://www.atmarkit.co.jp/ait/articles/1001/19/news106_2.html
/Users/azu/.ghq/github.com/asciidwango/js-primer/source/use-case/nodecli/refactor-and-unittest/README.md:72:5
                 v
    71. 
    72. ユニットテストを実行するには、Mochaが提供する`mocha`コマンドを使います。
    73. Mochaをインストールした後、package.jsonの`scripts`プロパティを次のように記述します。
                 ^

preset-ja-technical-writing/ja-no-redundant-expression: "実装を行う"は冗長な表現です。"実装する"など簡潔な表現にすると文章が明瞭になります。参考: http://www.atmarkit.co.jp/ait/articles/1001/19/news106_2.html
/Users/azu/.ghq/github.com/asciidwango/js-primer/source/use-case/todoapp/form-event/README.md:56:85
                                                                                                                                             v
    55. 
    56. 次のようにフォーム(`#js-form`)をEnterで送信すると、input要素(`#js-form-input`)の内容が開発者ツールのコンソールに表示されるという実装を行います。
    57. 
                                                                                                                                             ^

追記: "プログラムを実行"のような「カタカナする」系のものはデフォルトで無視するようにした。
textlint-ja/textlint-rule-ja-no-redundant-expression#15

@bot-user
Copy link

bot-user commented Dec 26, 2018

Deploy preview for js-primer ready!

Built with commit 66cddd1

https://deploy-preview-614--js-primer.netlify.com

@azu azu force-pushed the textlint-rule-ja-no-redundant-expression-update branch from 5945014 to b68aa43 Compare January 4, 2019 14:45
source/basic/implicit-coercion/README.md Outdated Show resolved Hide resolved
source/use-case/ajaxapp/display/README.md Outdated Show resolved Hide resolved
source/use-case/ajaxapp/promise/README.md Outdated Show resolved Hide resolved
source/basic/async/README.md Outdated Show resolved Hide resolved
source/basic/async/README.md Outdated Show resolved Hide resolved
source/basic/async/README.md Outdated Show resolved Hide resolved
source/basic/async/README.md Outdated Show resolved Hide resolved
source/basic/async/README.md Outdated Show resolved Hide resolved
source/basic/read-eval-print/README.md Outdated Show resolved Hide resolved
source/basic/string/README.md Outdated Show resolved Hide resolved
source/basic/string/README.md Outdated Show resolved Hide resolved
source/intro/feedback/README.md Outdated Show resolved Hide resolved
source/basic/condition/README.md Outdated Show resolved Hide resolved
@azu azu force-pushed the textlint-rule-ja-no-redundant-expression-update branch from 828a5d3 to d7d5b98 Compare January 8, 2019 14:00
@azu azu merged commit 97a413d into master Jan 12, 2019
@azu azu deleted the textlint-rule-ja-no-redundant-expression-update branch January 12, 2019 05:19
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