Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

Another attempt at upgrading to Babel 7 #59

Closed
wants to merge 6 commits into from

Conversation

nathanfriemel
Copy link

Upgraded all 'babel-' dependencies and various other dependencies that were still using babel 6. Then worked through the issues that came up:

Updated eslint to latest version that uses babel 7 and converted .eslintrc(s) from yaml to json.

Use strict now uses double quotes instead of single quotes; see this.

Ava was updated to a newer version that uses babel 7, it’s actually using rc1 since even version 0.25 was still using babel 6. For the tests this caused a change where the process.cwd() is now wherever the package.json is located when running tests; see this thread. This lead to changing all the tests config paths, removal of the ‘basic require with default webpack name’ test (for now, open to suggestions on how to re-add) and changing a lot of the expected fixture paths.

Expected fixtures are now double spaced to match generated output and paths that are replaced become double quoted, not sure why couldn’t find documentation on that one.

@codecov-io
Copy link

codecov-io commented Oct 23, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@cb486b7). Click here to learn what that means.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #59   +/-   ##
=========================================
  Coverage          ?   94.38%           
=========================================
  Files             ?        1           
  Lines             ?       89           
  Branches          ?        0           
=========================================
  Hits              ?       84           
  Misses            ?        5           
  Partials          ?        0
Impacted Files Coverage Δ
src/index.js 94.38% <100%> (ø)

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 cb486b7...c343ea4. Read the comment docs.

@nathanfriemel
Copy link
Author

Node.js 4 build is failing, according to this babel 7 has dropped support for Node.js < 6

@nathanfriemel
Copy link
Author

Closing this pull request because it's not getting any traction, I've also begun work on @babel/plugin-webpack-alias as a babel7 replacement over at https://github.com/shortminds/babel-plugin-webpack-alias if anyone is interested.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants