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.

Sign up for GitHub

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

Jump to bottom

Remove .babelrc Close #182 #216

Merged
merged 1 commit into from Feb 25, 2017
Merged

Remove .babelrc Close #182 #216

merged 1 commit into from Feb 25, 2017

Conversation

ykzts
Copy link
Collaborator

@ykzts ykzts commented Feb 25, 2017 •
edited

.babelrcを削除して、webpackから使われるものはwebpack.config.jsに、Jestから使われるものはpackage.jsonにそれぞれBabelのオプションを記述する。これはビルド用のものはビルド用のものと職域が明確になるようにするためのものである。

関連Issue

@ykzts
Remove .babelrc Close #182
5b5d38d 
`.babelrc`を削除して、webpackから使われるものは`webpack.config.js`に、
Jestから使われるものは`package.json`にそれぞれBabelのオプションを記述する。
これはビルド用のものはビルド用のものと職域が明確になるようにするためのものである。
@ykzts ykzts added this to the 2.0.0 milestone Feb 25, 2017
@ykzts ykzts self-assigned this Feb 25, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+0.5%) to 15.789% when pulling 5b5d38d on rm-babelrc into ec16015 on master.

@ykzts ykzts merged commit 54bb160 into master Feb 25, 2017
@ykzts ykzts deleted the rm-babelrc branch February 25, 2017 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ykzts ykzts

Labels
enhancement
Projects
None yet
Milestone
2.0.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ykzts @coveralls