Soichiro Miki

Results 48 comments of Soichiro Miki

@jakubdrozdek That happened on our fork (Japanese), too. That [dawidd6/action-download-artifact@v2](https://github.com/dawidd6/action-download-artifact) GitHub action tries to download "external" files (artifact) that had been generated by a workflow executed on the `main` branch....

The current pre-commit hook has the following two lines: https://github.com/reactjs/reactjs.org/blob/4022f9774b8dd93d863fd6bca6447b1f03ae3979/beta/.husky/pre-commit#L5-L6 This may make sense on the main repo but makes no sense on translation forks. Every time I commit, I...

@gaearon In the future, it would be nice to run pre-commit generate-ids only on the main repo. When it finds a heading without an explicit ID, it should reject the...

@gaearon Please set up the Japanese beta docs, too. Translations for several pages and MDX tags were merged into main.

@gaearon `Analyze bundle` job still fails for every new PR made in translation forks. Looks like the cause is this (Line 57): https://github.com/reactjs/reactjs.org/blob/c005fa4d7a2a0a602d6d538526699d7073ae3109/.github/workflows/analyze.yml#L53-L60 Line 57 refers to `bundle_analysis_upload.yml`, but this...

I made a PR to fix the problem above: #4192

What will the translation process be like? The current [translation bot](https://github.com/reactjs/reactjs.org-translation/tree/master/scripts) merges everything from the original repo's `master` branch into each translation repo, but it sees only the `master` branch....

12/22現在、以下の2点が未解決です。 - **PRを作成するとあらゆるCIがこける**: 原因をつきとめたので upstream で修正済み、こちらへの取り込み待ち (reactjs/reactjs.org#4192) - **textlint のセットアップ**: サブディレクトリに husky で pre-commit フックをセットアップした時に何らかの環境依存でエラーが起こっており、それの修正が必要。 #454 これらが解決したら本格的に全記事の翻訳にとりかかっていいかなと思います。

beta ディレクトリの中はリポジトリのトップレベルとは独立した package.json や yarn.lock を持つ別環境であり、beta ディレクトリの中と外の世界(現行サイト)が互いに干渉しないことになっているように見えます。変則的な monorepo と言いますか。現行サイトは Gatsby で新サイトは Next.js ですし。 将来的には beta の中身がそっくりそのままリポジトリのルートを置き換え、現行のリポジトリルートにある(beta 以外の)内容はどこかにアーカイブされます。逆に、現在のリポジトリのルートにあるコンテンツやスクリプトはあくまで現行のドキュメントサイトを担当しており、betaディレクトリの中身について一切関知してはいけません。 このPRでは、現行サイトを構築するためのスクリプトが beta の存在に依存してしまっていますし、将来 beta がメインサイトとして昇格する際にも問題が生じてしまいます。面倒でも、beta という別環境に、現行の textlint と同様のセットアップを再現する必要があると思います。`textlint/jp-reactjs-org-lint.js` もそのまま beta 内にコピーする感じですね。

言い方を変えると、クローンした後に beta ディレクトリだけ全く別の場所に動かしたとしても、現行サイトは beta なしで完璧にビルドできる必要がありますし、beta は beta で親フォルダにあったファイルへの依存なしに完璧にビルドできる必要があります。