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

Use babel-register script when running babel smoke tests #442

Merged
merged 1 commit into from
Mar 30, 2017
Merged

Conversation

existentialism
Copy link
Member

@existentialism existentialism commented Mar 30, 2017

Q A
Bug fix? yes
Breaking change? no
New feature? no
Deprecations? no
Spec compliancy? no
Tests added/pass? yes/no
Fixed tickets
License MIT

When running the babel smoke test, the recent change to use --compilers js:babel-register is causing test failures as it's attempting to use babylon's babelrc:

TypeError: [BABEL] /home/travis/build/babel/babylon/node_modules/istanbul-lib-instrument/dist/index.js: Plugin 0 specified in "/home/travis/build/babel/babylon/.babelrc.env.test" was expected to return a function but returned "undefined" (While processing preset: "/home/travis/build/babel/babylon/node_modules/babel-preset-stage-0/lib/index.js") (While processing preset: "/home/travis/build/babel/babylon/node_modules/babel-preset-es2015/lib/index.js")

@codecov
Copy link

codecov bot commented Mar 30, 2017

Codecov Report

Merging #442 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #442   +/-   ##
=======================================
  Coverage   98.25%   98.25%           
=======================================
  Files          20       20           
  Lines        3504     3504           
  Branches      927      927           
=======================================
  Hits         3443     3443           
  Misses         22       22           
  Partials       39       39
Flag Coverage Δ
#babel 82.02% <ø> (ø) ⬆️
#babylon 97% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2f3123c...ec6498a. Read the comment docs.

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

2 participants