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

Backport: use preset-env and remove flow-strip-types #324

Merged
merged 3 commits into from
May 20, 2017

Conversation

yavorsky
Copy link
Member

Nothing special, just use preset-env for backport and remove flow-strip-types plugin.

@codecov-io
Copy link

codecov-io commented May 15, 2017

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #324   +/-   ##
======================================
  Coverage    93.2%   93.2%           
======================================
  Files           4       4           
  Lines         206     206           
  Branches       60      60           
======================================
  Hits          192     192           
  Misses          9       9           
  Partials        5       5

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 fd35519...79d8e72. Read the comment docs.

@hzoo
Copy link
Member

hzoo commented May 19, 2017

Is this necessary to backport though?

@yavorsky
Copy link
Member Author

@hzoo For now it's still the main branch and few people asked me why we are using es2015 instead of env 😕

package.json Outdated
"loose": true
}
]
["env"]
Copy link

Choose a reason for hiding this comment

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

Isn't it better to use the current code instead of always the previous version?
So replacing ["env"] with "./src"?

Copy link
Member

Choose a reason for hiding this comment

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

If you have a bug then it won't build itself

@hzoo
Copy link
Member

hzoo commented May 19, 2017

@yavorsky lol really..

@existentialism existentialism merged commit 6bc5326 into babel:master May 20, 2017
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants