Skip to content

Conversation

t0yohei
Copy link
Contributor

@t0yohei t0yohei commented Mar 20, 2022

resolve #41

よろしくお願いします。

- [Vite deployment guide](https://vitejs.dev/guide/static-deploy.html)
- [Vue CLI deployment guide](https://cli.vuejs.org/guide/deployment.html)
- [Vite 本番環境用のビルドガイド](https://ja.vitejs.dev/guide/build.html)
- [Vite デプロイガイド](https://ja.vitejs.dev/guide/static-deploy.html)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

[memo]
Vite は日本語の翻訳があったので、リンク先を日本語のものにしました。

Copy link
Member

@jay-es jay-es left a comment

Choose a reason for hiding this comment

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

レビューしました。
1点修正お願いします:pray:

## ビルドツールあり

Projects scaffolded via `create-vue` (based on Vite) or Vue CLI (based on webpack) are pre-configured for production builds.
`create-vue` (Vite ベース) または Vue CLI (webpack ベース) で足場が組まれたプロジェクトは、本番ビルド用にあらかじめ設定がされています。
Copy link
Member

Choose a reason for hiding this comment

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

scaffold はそのままだと伝わりにくそうなので「生成」にすると良いかなと思いました

Suggested change
`create-vue` (Vite ベース) または Vue CLI (webpack ベース) で足場が組まれたプロジェクトは、本番ビルド用にあらかじめ設定がされています。
`create-vue` (Vite ベース) または Vue CLI (webpack ベース) で生成されたプロジェクトは、本番ビルド用にあらかじめ設定がされています。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

レビューありがとうございます、修正しました!
ca53df7

Copy link
Member

@kazupon kazupon left a comment

Choose a reason for hiding this comment

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

みました!
jay-es さん以外、特に問題ありそうなところはありませんでした。

Copy link
Member

@jay-es jay-es left a comment

Choose a reason for hiding this comment

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

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

@jay-es jay-es merged commit 712be56 into vuejs-translations:main Mar 22, 2022
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.

「Guide > Best Practices > Production Deployment」の翻訳

3 participants