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

Add @babel/plugin-transform-flow-strip-types in favor of @babel/preset-flow #505

Merged
merged 2 commits into from
Sep 5, 2018

Conversation

KevinGrandon
Copy link
Contributor

@KevinGrandon KevinGrandon commented Sep 5, 2018

  • Installs and uses @babel/plugin-transform-flow-strip-types.
  • Introduces a test case for a failure after upgrading to [email protected].

Note: We were including @babel/preset-flow inside of our get-babel-config.js file, but it seems to not actually work. It seems that this is due to some other plugins running before the preset, possibly @babel/plugin-proposal-class-properties. We have to unshift the plugin to strip flow types first. This means that we can remove the @babel/preset-flow plugin.

This bug was introduced from #486

@KevinGrandon KevinGrandon force-pushed the add-class-props-test branch 2 times, most recently from 90b039f to bcac52a Compare September 5, 2018 21:29
@KevinGrandon KevinGrandon changed the title Add failing test case for transformed class properties /w flow class annotations Add @babel/plugin-transform-flow-strip-types Sep 5, 2018
@KevinGrandon KevinGrandon changed the title Add @babel/plugin-transform-flow-strip-types Add @babel/plugin-transform-flow-strip-types in favor of @babel/preset-flow Sep 5, 2018
@KevinGrandon
Copy link
Contributor Author

!merge

@old-fusion-bot old-fusion-bot bot merged commit 09b075a into fusionjs:master Sep 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants