-
Notifications
You must be signed in to change notification settings - Fork 87
Guide> Production Deployment の翻訳 #270
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
Conversation
Deploy preview for vuejs-v3-ja-doc-preview ready! Built with commit c5f4be3 https://deploy-preview-270--vuejs-v3-ja-doc-preview.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
レビューしました!
確認お願いします!
|
||
### ビルドツールあり | ||
|
||
Webpack や Browserify などのビルドツールを使う場合は、プロダクションモードは Vue のソースコード内の `process.env.NODE_ENV` で決定され、デフォルトでは開発モードになります。どちらのビルドツールも、 Vue のプロダクションモードを有効にするために、この変数を上書きする方法を提供しており、警告はビルド中に Minifier (圧縮・軽量化) によって取り除かれます。Vue CLI では事前設定されていますが、どのように行われているか知っておくことはよいでしょう: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
独り言:
原文も Browserify
って書いてあるのか。(今、Vue アプリケーションを Browserify でバンドルしているのないと思うんだけど... vueify もメンテしてないし https://github.com/vuejs/vueify)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
自分も翻訳していて、まだ Browserify?って思っちゃいました。
バンドラーもメンテしてないんですね。これはちょと本家に提案をコミットするチャンスかな。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
そうですね。これはコミットチャンスですね。
結構、大きく変わる部分なので、英語でやり取りが大変だと思いますが。
Browserify の代わりの載せるとしたら、Vite でしょうかね。
} | ||
``` | ||
|
||
#### Browserify |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
独り言:
あれ、Browsery についてまだ載せてる。
何か意図があるのかな。 🤔
Co-authored-by: kazuya kawaguchi <[email protected]>
レビューポイントの反映しました |
Description of Problem
Guide > Production Deployment の翻訳を追加しました。
https://v3.vuejs.org/guide/tooling/deployment.html
https://github.com/vuejs/docs-next/blob/master/src/guide/tooling/deployment.md
Proposed Solution
対象ファイルが存在しなかったため、新規の作成です。
Additional Information
カタカナのままがよいものが多くなってしまったので、ツッコミ大歓迎です。