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.

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

Fix Karma config to babelify node_modules too #3173

Merged
merged 1 commit into from
Feb 1, 2017

Conversation

bajtos
Copy link
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

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

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