Skip to content

Conversation

@dsyme
Copy link
Contributor

@dsyme dsyme commented Feb 20, 2019

Testing to follow

@dsyme
Copy link
Contributor Author

dsyme commented Feb 20, 2019

Tests added, this is now ready. It wasn't hard change

@dsyme
Copy link
Contributor Author

dsyme commented Feb 20, 2019

I've enabled ExprTests.fs in VisualFSharp.Unittests.dll - that was disabled for some reason - we should find out why from test failures

@dsyme
Copy link
Contributor Author

dsyme commented Feb 20, 2019

FCS docs also updated

@cartermp
Copy link
Contributor

Linking #6189

@dsyme
Copy link
Contributor Author

dsyme commented Feb 21, 2019

OK this is ready

Copy link
Contributor

@cartermp cartermp left a comment

Choose a reason for hiding this comment

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

Change looks good and simple (looks like there's a bit of test file cleanup as well).

@cartermp
Copy link
Contributor

@alfonsogarciacaro Does this look good enough? I imagine so, since Fable code just works off of the types that have been adjusted.

@baronfel
Copy link
Member

Once this gets merged I can do a merge+release of FCS to support Fable.

@dsyme
Copy link
Contributor Author

dsyme commented Feb 21, 2019

@cartermp I think it's all the info we're able to give :-)

@alfonsogarciacaro
Copy link
Contributor

Fable REPL (Try a hard reload to clear the browser cache if it doesn't work) 😸

@cartermp cartermp merged commit b0292b8 into dotnet:master Feb 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants