Skip to content

docs: update docs/ja #394

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

Merged
merged 6 commits into from
Feb 3, 2018
Merged

docs: update docs/ja #394

merged 6 commits into from
Feb 3, 2018

Conversation

38elements
Copy link
Contributor

No description provided.

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.

レビュー 👀 コメント 💬 しました。
確認お願いします。


このファイルは Karma を設定するために使用されます。

ファイルを Webpack で前処理する必要があります。そのために、 Webpack をプレプロセッサとして加えます。そして、 Webpack の設定を含めます。プロジェクトに元々あった Webpack の設定ファイルを変更なしで使用することができます。
Copy link
Member

Choose a reason for hiding this comment

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

preprocessorプリプロセッサの訳が一般的なような気がします。


ファイルを Webpack で前処理する必要があります。そのために、 Webpack をプレプロセッサとして加えます。そして、 Webpack の設定を含めます。プロジェクトに元々あった Webpack の設定ファイルを変更なしで使用することができます。

この設定では、 Chrome でテストを実行します。他のブラウザーを加える方法は [Karma のドキュメントにある Browsers セクション](http://karma-runner.github.io/2.0/config/browsers.html) を見てください。
Copy link
Member

Choose a reason for hiding this comment

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

ブラウザという訳が一般的なような気がします。

`Counter.vue` ファイルを `src` ディレクトリに作成します。

``` html
<template>
Copy link
Member

Choose a reason for hiding this comment

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

SFCの<template>ブロックのインデントが気になりました。なるべく公式スタイルガイドに合わせて一貫性をもたせた方がいいかなと思います。enなどの他の言語も同様です。

npm install --save-dev karma-coverage cross-env
```

環境変数の `BABEL_ENV` をセットするために `cross-env` を使います。テストをコンパイルする時に babel-plugin-istanbul を使用します。プロダクションコードをコンパイルする時は `babel-plugin-istnabul` を含めるべきではありません。
Copy link
Member

Choose a reason for hiding this comment

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

babel-plugin-istanbulが原文と異なりバッククォートされていないようです。

@38elements
Copy link
Contributor Author

@kazupon
レビューありがとうございます。
指摘していただいた点を修正しました。
よろしくお願いします。

@eddyerburgh eddyerburgh merged commit f8ce240 into vuejs:dev Feb 3, 2018
@38elements 38elements deleted the document branch February 6, 2018 12:28
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.

3 participants