forked from dotnet/fsharp
-
Notifications
You must be signed in to change notification settings - Fork 0
Forki i1637 #25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Forki i1637 #25
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Don't convert attributes to array and then to list * Don't convert attributes to array and then to list
Adding features since last update.
…otnet#3150) * Give better error message when IF Statement has context requirements * simplify code * Update TypeChecker.fs * something has a type * Adding a test * Transitive IF context * Another test - fixes dotnet#3146 * fix wording
…tection Better indexer detection
…sing DisableImplicitFSharpCoreReference (dotnet#3166)
Try to clean up error text a little.
* remove portables * temp * remove bin from profiles paths * reorder setup build * Ensure nuget is built prior to setup * remove portable customizations from targets * fix ci_part1
…otnet#3195) * Simplify ci to 3 parts now that we don't build many fsharp.core.dlls * ci_part1 was still to broad
There's several reasons for this, the most important being that only .NetFramework projects will provide a useful download. Also, not doing this resulted in a bug for Xamarin projects (Issue 2992). This fix is identical to what other projects do in Visual Studio.
This fixes the very long-standing issue where trying to add a new file to an F# Web project opens a window with no file templates available. See for example here for the workaround that was required for VS up to 2015: http://bloggemdano.blogspot.fr/2013/11/adding-new-items-to-pure-f-aspnet.html
* Struct Records and Unions have ValueType super type * Add qa tests for ValueType constraint on struct Records and Unions
Add determinism by @davidglassborow
* Increase indent level after line end with certain keywords * Fix test case * Use Tokenizer * Refactoring and fix test * Saul review * Add more test * Fix tests * Rename tryFindLastNoneEmptyToken -> tryFindLastNoneWhitespaceToken * Trailing comment is ok * Move auto indent to a separate test * Use same indentation as the previous comment line
…net#3224) * fix 1510: Static constructor interferes with looking up ReflectedDefinition * fix 1510: Static constructor interferes with looking up ReflectedDefinition (2) * fix test
* remove duplicated completion items * remove duplicated completion items (2)
* Fix quotation of struct union match * Fix quotation of struct union match () * fix test
* Add fsharp.core build to proto * update linux * update linux build script
…pe providers (dotnet#3236) * no in-memory project references for projects using generative type providers * add test and build FCS as part of 'all' * remove needless file * add missing compiled TP dll
* Add dotnetcli20 * Update dotnet cli * Build proto using dotnet cli as lkg * Remove lkg
* improve VS perf * improve VS perf (2)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updat to master and also remove change to TAST dsyme@31c8082