Translation foundation system structure

[potato4d]

This project is a part of #2

Changes:

• Install TextLint
• Install TextLint JTF-style preset
• Format documents w/ TextLint auto-formatter and manual work
• Add lint:text to all-check and ci-check target

[potato4d]

CircleCI does not seem to be valid yet.
I will turn it on.

[potato4d]

I confirmed that CI will fail. After correction, remove WIP.

[smikitky]

Maybe we can ignore/exclude untranslated materials using something like glob or .XXXignore? Looks like many errors detected by textlint are ones that should be handled in the upstream repo.

[potato4d]

Looks like many errors detected by textlint are ones that should be handled in the upstream repo.

I agree about many cases.

For example, the following should be processed with upstream.

https://github.com/reactjs/ja.reactjs.org/pull/6/files#diff-b01ab259175317c1784e1ece8d505291R183

On the other hand, There is also a problem dedicated to Japanese documents.

https://github.com/reactjs/ja.reactjs.org/pull/6/files#diff-c41f5967bd8994ce9a1935f0ef2488d5R283

But, as far as I know, TextLint is not have ignore file or filter system.
Alternative: Introducing TextLint incrementally using lint-staged

[potato4d]

memo: 一旦 JTF-Style から不必要なものを省いたが、徐々にカスタムルールを設定していく必要がある（ルールを書くのには多少時間がかかるので、ひとまずあとで行う）

[smikitky]

Probably it's good to submit a separate PR to the upstream repo that fixes those mistakes in the original source files. Eventually the fix will be merged here and the problem will be gone (at least for a while).

[potato4d]

@smikitky
ちょっと混乱してきたので日本語で申し訳ないのですが、 upstream はオリジナル文書という認識ですかね？

upstream(en) / fork(ja) の場合、 JTF-Style はあくまでも日本語の取扱についての規則なので、 < を < にするみたいな個別事項は Pull Request を送ってもよさそうな反面、全角・半角の取扱などは ja の都合を en に押し付けるのは微妙かなぁと思っています。

なので個人的にはこれはこれで入れてしまって、その上で < を < にといった作業は個別で Pull Request を reactjs.org に送るのが丸いかな。という印象です。

ja 翻訳のスピード感都合で少し視点が歪んでるかもしれませんが、どうでしょう？

[smikitky]

はい、upstream はフォーク元のリポジトリ（今回は reactjs/reactjs.org）です。

改めて眺めてみると、確かに未翻訳の原文の改変だけ見ても、「これは確かに原文自体が完全にミス」というのと「これは textlint の都合であって本来どうでもいい」というのと「むしろ textlint のミスであって原文が正しい」というのが混在していますね…。blog/2014-02-05-community-roundup-15.md のとかは原文が正しいような気がします。

当面 blog をチェックから外すのも難しい感じですか？

[potato4d]

@smikitky
glob 記法が使えるので blog を限定的に外すみたいなことは可能ですね。そうしましょうかー。

[potato4d]

@smikitky
Just completed these.

• rollback content/blog content same as master
• Ignore lint target to content/blog

I'm waiting for your review!

[smikitky]

当面 blog だけではなく community の中身も lint かけずに放置の方がよい気がします。やるとしても数週間は先でしょうし。

[smikitky]

あとは community の中で改変された韓国語や中国語のファイルをロールバックする必要がありそうですね

[smikitky]

あと使い方なんですが yarn した後に npm run lint:text するだけでいいんでしょうか、他に設定とか準備とかありますか？ (TRANSLATION.md に書かないといけないので）

[potato4d]

@smikitky

devdependencies に入っているので npm run lint:text だけで大丈夫です。
今回一緒にやってしまうか悩みましたが、 現行の "lint": "eslint .", を "lint:eslint": "eslint .", にして、 "lint": "npm-run-all -p lint:*", としたほうが翻訳者には優しいかなと思いました。

npm run lint で全てまとめて(かつ並列で)走ってくれるので

[potato4d]

TRANSLATION.md は TRANSLATION.md で書いたほうがよさそう(用語集を追加する時に)だと考えてここでは編集していませんが、この PR でまとめて書いてしまいますか？

[smikitky]

TRANSLATION.md は別にこちらで編集しようかなと思います。（ちょっと1時間や2時間ではできないですが）

lint:text を動かして色々エラー入れ込んで確かめましたが、いい感じで動いてますね、よいと思います！

[smikitky]

_{マージはお任せします}

[potato4d]

ありがとうございます！
セルフマージはあんまり健全じゃなさそうなので、基本はマージしてもらう形のほうが気持ちはありがたいかもしれません 🙏

[smikitky]

次からそうします！