reactjs / ja.react.dev

React documentation website in Japanese

Home Page:https://ja.react.dev

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

散見される`。(`の修正をしてもよいでしょうか。

kobaji opened this issue · comments

初投稿です。Reactの勉強をしていて読みづらい箇所を見つけ、以下を修正したいです。

https://github.com/reactjs/ja.reactjs.org/blob/master/content/home/examples/a-stateful-component.md#7 などで、不適切な。(の使われ方をしているように見えます。

できます。(これは this.state で参照することができます)状態

これは、「できます(これは this.state で参照することができます)。状態」が適切だと思われます(事実英語ではxxx(xxx).xxxなっています)。これを修正したいですがよろしいでしょうか。

masterブランチに対して。(でGREPすると35件ヒットし、そのうちのいくつかは同様に修正したいですがいかがでしょうか。

確かに読みづらいですね、すみません…。

うろ覚えなのですが、日本語のミスを減らすために導入したはずの textlint のJTFプラグインの過剰検出のため、仕方なくこうなっていたような気がします。

git commit .しようとすると自動で textlint が走ってエラーが出るはずですので、試しに1カ所修正してコミットしてみて普通にコミット可能かどうか確認していただいても良いでしょうか。以下のファイルが設定なので、そこで単純に検出オプションを1個無効にする程度で済めばそのようにするのが良いと思います。

https://github.com/reactjs/ja.reactjs.org/blob/master/.textlintrc

もし厳しいようでしたら言っていただければそのうちこちらで対応します。

ご連絡ありがとうございます。年が明けたらトライしてみます。

@smikitky さんの予想とは異なり、git commitしてもエラーが起こりませんでした。このまま全箇所について修正をかけてもよろしいでしょうか。

詳細:
kobaji@31a91ec
.textlintrc を変更せず、()。の使い方を1ヶ所修正してみました。特にエラーは出ていないようですね。
yarnのテストも行なってみようと思いましたが、ネットワークの都合でYarnのインストールができずテストできていません。

はい、では修正していただいて大丈夫です。お手数をおかけします。