Add support for flow type spread in 0.43 #1064

Merged
merged 1 commit into from Apr 4, 2017

Conversation

Projects
None yet
2 participants
@vjeux
Collaborator

vjeux commented Mar 21, 2017

@vjeux

This comment has been minimized.

Show comment
Hide comment
@vjeux

vjeux Mar 21, 2017

Collaborator

Here's the (not merged yet) PR on babylon babel/babylon#418

Collaborator

vjeux commented Mar 21, 2017

Here's the (not merged yet) PR on babylon babel/babylon#418

@vjeux vjeux changed the title from Add support for flow type spread in 0.42 to Add support for flow type spread in 0.43 Apr 4, 2017

@vjeux

This comment has been minimized.

Show comment
Hide comment
@vjeux

vjeux Apr 4, 2017

Collaborator

Now that we are on babylon beta8 (thanks @existentialism), it also works for babylon!

Collaborator

vjeux commented Apr 4, 2017

Now that we are on babylon beta8 (thanks @existentialism), it also works for babylon!

@vjeux vjeux merged commit 1578e18 into prettier:master Apr 4, 2017

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@@ -0,0 +1,2 @@
+run_spec(__dirname);
+run_spec(__dirname, {parser: 'flow'});

This comment has been minimized.

@existentialism

existentialism Apr 4, 2017

Collaborator

Should be parser: "babylon" since default is flow, yeah?

@existentialism

existentialism Apr 4, 2017

Collaborator

Should be parser: "babylon" since default is flow, yeah?

This comment has been minimized.

@vjeux

vjeux Apr 4, 2017

Collaborator

Woops, good call! I'll fix this :) I tested it on the command line and it's working on both.

@vjeux

vjeux Apr 4, 2017

Collaborator

Woops, good call! I'll fix this :) I tested it on the command line and it's working on both.

vjeux added a commit to vjeux/prettier that referenced this pull request Apr 4, 2017

@vjeux vjeux referenced this pull request Apr 4, 2017

Merged

Fix spread snapshot test #1119

vjeux added a commit that referenced this pull request Apr 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment