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

docs: remove static optimization indicator #190

Conversation

numb86
Copy link
Contributor

@numb86 numb86 commented Jan 24, 2021

「静的最適化インジケータ」ページの状態を本家に追従させるための PR です。

本家では現在、マークダウンファイルは存在するものの、docs/manifest.jsonから削除され、公式サイトからは閲覧できない状態になっています。
また、関連項目が削除され、注意書きが追加されています。
https://github.com/vercel/next.js/blob/5baedf55/docs/api-reference/next.config.js/static-optimization-indicator.md

docs/manifest.jsonから削除される際に、Automatic Static Optimizationからのリンクも削除されています。
https://github.com/vercel/next.js/blob/5baedf55/docs/advanced-features/automatic-static-optimization.md

この PR で、本家と同じ状態になります。

cf. vercel/next.js#18629

Copy link
Member

@Shinyaigeek Shinyaigeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます!!!

docs/manifest.json 以外は良さそうです!

docs/manifest.json については, 3日前に Timがまた static optimization indicator の項目を前とは別の場所に追加するコミットをしていますね 🤔 (vercel/next.js@d991e3c)

まだTimのコミットの本番反映はされてないので, このPRの状態を最新として取り込んでもいい気がしますし, 先んじてTimのコミットも反映してしまうのもいいかと思いますが, どうしましょうか?

@Shinyaigeek Shinyaigeek requested a review from 5t111111 as a code owner January 24, 2021 06:44
@numb86
Copy link
Contributor Author

numb86 commented Jan 24, 2021

docs/manifest.json については, 3日前に Timがまた static optimization indicator の項目を前とは別の場所に追加するコミットをしていますね🤔

全く気付きませんでした……。Issue でも議論されてますが、本家に追従していくのはかなり大変そうですね。
私としては、どちらの対応でも構わないです。

Copy link
Member

@Shinyaigeek Shinyaigeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

僕としては本家ドキュメントの内本番反映されている分だけ日本語ドキュメントに反映させる方がユーザーからすれば混乱が少なくていいかなと思ったので, ひとまずTimのコミットは無視して docs/manifest.json についてもこのまま取り込むでいいかなと感じたので, Approveさせていただきます 👍

他のレビュアーの方も何か意見あれば聞きたいです 🙏

@yokinist
Copy link
Contributor

PRありがとうございます!

ここについては翻訳済ドキュメントの追従方針がまだ決まっていなかったので #189 にもイシューをたてていたところでした!
なので、まずは追従方針(特に今回の場合はタイミング)について決める必要があると思っています!

個人的には気付いたときに随時追従/更新する方針だと、きりがなく運用が大変そうなのと今回のように今後変わりうる可能性も高いものもあるので、基本的には新しいマイナーバージョン(例えば次だったら v10.1.0 など)が出たタイミングに絞り追従させる方針がいいのかなと思いました。(先日更新させた #186 は方針決まっておらず、その時点でのcanaryを追従させてしまいました...🙇‍♂️)

あくまで判断材料としてですが、マイナーバージョンのリリースタイミングに絞るとなると現時点で最新が v10.0.6-canary.7 になるので追従先は v10.0.0 になりそうです(既に反映してしまったのと以前の更新もリリース時のものでなかったので、もしこの運用でやるなら次リリースから運用したいと思います)。

https://github.com/vercel/next.js/tree/118ab7992bc8f7a7e5a7bb996510d9b56ffe4f68

何かアイデアあればコメントいただけると嬉しいです 🙏

Copy link
Collaborator

@5t111111 5t111111 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

明快な方針が決まるまでは、タイミングによってアドホックな対応になるのは仕方なしとした上で、

・本家の本番で公開されている内容にできるだけあわせる

とするのが良いかと思いました。

したがって、 @Shinyaigeek さんの

僕としては本家ドキュメントの内本番反映されている分だけ日本語ドキュメントに反映させる方がユーザーからすれば混乱が少なくていいかなと思ったので, ひとまずTimのコミットは無視して docs/manifest.json についてもこのまま取り込むでいいかなと感じたので, Approveさせていただきます 👍

と同意見で、 現時点でまだ本番公開はされていないため approve させてもらいます!

@Shinyaigeek Shinyaigeek merged commit 46d567f into Nextjs-ja-translation:master Feb 20, 2021
@numb86 numb86 deleted the update-static-optimization-indicator branch February 21, 2021 04:45
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.

4 participants