Skip to content

Fix Karma config to babelify node_modules too#3173

Merged
bajtos merged 1 commit intomasterfrom
fix/strong-remoting-in-karma
Feb 1, 2017
Merged

Fix Karma config to babelify node_modules too#3173
bajtos merged 1 commit intomasterfrom
fix/strong-remoting-in-karma

Conversation

@bajtos
Copy link
Copy Markdown
Member

@bajtos bajtos commented Feb 1, 2017

Before this change, dependencies in node_modules (e.g. strong-remoting)
were not transformed to ES5 and thus crashed the tests in PhantomJS.

Note that loopback-datasource-juggler cannot be babelified to ES5
because it does not correctly support strict mode yet.

See also #3003

@bajtos bajtos self-assigned this Feb 1, 2017
@bajtos bajtos added the review label Feb 1, 2017
@coveralls
Copy link
Copy Markdown

coveralls commented Feb 1, 2017

Coverage Status

Coverage remained the same at 89.056% when pulling 96a4892 on fix/strong-remoting-in-karma into 76dd35e on master.

Before this change, dependencies in node_modules (e.g. strong-remoting)
were not transformed to ES5 and thus crashed the tests in PhantomJS.

Note that loopback-datasource-juggler cannot be babelified to ES5
because it does not correctly support strict mode yet.
@bajtos bajtos force-pushed the fix/strong-remoting-in-karma branch from 96a4892 to 3c209ee Compare February 1, 2017 11:27
@coveralls
Copy link
Copy Markdown

coveralls commented Feb 1, 2017

Coverage Status

Coverage remained the same at 89.056% when pulling 3c209ee on fix/strong-remoting-in-karma into 76dd35e on master.

@bajtos bajtos merged commit db3c7b6 into master Feb 1, 2017
@bajtos bajtos deleted the fix/strong-remoting-in-karma branch February 1, 2017 11:46
@bajtos bajtos removed the review label Feb 1, 2017
@bajtos bajtos mentioned this pull request Feb 1, 2017
2 tasks
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.

2 participants