Skip to content

Conversation

@dsyme
Copy link
Contributor

@dsyme dsyme commented Feb 13, 2018

It is possible that we should revert #4173 until we understand the issues better and to ensure stability between F# 4.1 and its update releases.

See customer-reported issue #4343

This is trialling a revert of combined commit f9893b6.

@dsyme
Copy link
Contributor Author

dsyme commented Feb 13, 2018

@brettfo Close this since we have #4348 ? I assume you integrate dev15.6 --> master?

@dsyme dsyme closed this Feb 13, 2018
@dsyme dsyme reopened this Feb 13, 2018
@dsyme
Copy link
Contributor Author

dsyme commented Feb 13, 2018

@brettfo Is it OK if we apply this to the master branch too? I just want the core compiler logic to be in sync, so we can integrate it to Mono etc. (Mono release repo http://github.com/fsharp/fsharp is formed by integrating "master" branch here)

@brettfo
Copy link
Member

brettfo commented Feb 13, 2018

Yes we plan on merging dev15.6 back into master, but it might be a few weeks so having this separate PR is probably a good idea.

@dsyme
Copy link
Contributor Author

dsyme commented Feb 13, 2018

@brettfo Thanks. Merging since it was green before I opened it

@dsyme dsyme merged commit 7377b02 into dotnet:master Feb 13, 2018
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.

2 participants