Skip to content
This repository has been archived by the owner on Jan 19, 2021. It is now read-only.

docs(README): add example for babel usage #56

Merged
merged 2 commits into from
Jun 27, 2017

Conversation

tarjei
Copy link
Contributor

@tarjei tarjei commented Apr 26, 2017

No description provided.

@jsf-clabot
Copy link

jsf-clabot commented Apr 26, 2017

CLA assistant check
All committers have signed the CLA.

README.md Outdated
{
test: /\.js$|\.jsx$/,
enforce: 'post',
loader: 'istanbul-instrumenter-loader',
Copy link
Member

Choose a reason for hiding this comment

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

Please update to webpack >= v2.0.0 syntax here

+ use: { loader: 'istanbul-instrumenter-loader', options: {...} }
- loader: ...
- query: ...

README.md Outdated
@@ -91,6 +91,21 @@ config.set({
});
```

#### Webpack 2.0 with Babel
Copy link
Member

Choose a reason for hiding this comment

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

### `with babel`

@michael-ciniawsky michael-ciniawsky changed the title Add example for usage with Babel docs(README): add example for babel usage Jun 26, 2017
@michael-ciniawsky michael-ciniawsky added this to the 3.0.1 milestone Jun 26, 2017
@michael-ciniawsky michael-ciniawsky self-assigned this Jun 26, 2017
@tarjei
Copy link
Contributor Author

tarjei commented Jun 27, 2017

@michael-ciniawsky Thanks for the review!

I changed the options: {...} in your suggestion to options: { esModules:true }.

But: I'm not sure if the esModules: true is needed with babel. It is a long time since I wrote this PR...

Copy link
Member

@michael-ciniawsky michael-ciniawsky left a comment

Choose a reason for hiding this comment

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

@tarjei thx

@michael-ciniawsky michael-ciniawsky merged commit 1897db1 into webpack-contrib:master Jun 27, 2017
@tarjei tarjei deleted the patch-1 branch July 26, 2017 11:48
@michael-ciniawsky michael-ciniawsky removed this from the 3.0.2 milestone Oct 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants