forked from dotnet/fsharp
-
Notifications
You must be signed in to change notification settings - Fork 0
merge from master #17
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
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
* Update Tutorial Script * Feedback and add more print statements
* Show that we can't suggest record labels * Suggest record labels - fixes #2117 * cleanup * Fix the test * New test for union cases * Suggest union cases * Suggest more unions * Fix tests
* Insert closing brackets if they are missing * Find correct interface identifiers if object expressions are on the same lines with let bindings
* Add a service for XmlDocParser * Add corresponding VS service * Use VS workspace to retrieve correct documents * Use current snapshot to ensure source text is fresh
* Propose Uppercase for FS0053 * make ProposeUppercaseLabel code fix work on all symbol uses * cleanup * refactor with asyncMaybe * add ParseAndCheckDocument * refactor ProposeUppercaseLabel
* return the union's tag when case is nullary * fix IL tests * ensure that gethashcode is not inlined * only for compilergenerated cases now
* fix 1962 * fix warnings * simplify diagnostics processing * simplify diagnostics processing (2) * remove project * fix build * fix build * fix build * fix build * merge master * fix build of tests * fix error flag * fix tests and build * fix tests * fix test * test diagnostic * fix isError mistake and clarify namings Error --> Diagnostic
* add FSharpDeclarationListItem.IsAttribute try to handle attributes in a special way at attribute application point (wip) * almost works * suggest all types, modules and namespaces at attribute application position * fix Context.AttributeApplication detection * autocomplete does not remove "Attribute" suffix if an attribute type does not have it fix related tests * do not try to use AstVisitorBase to determine that we are at attribute application position * try to fix tests * fix some tests * cut attribute prefix on editor side * fixed: IsAttribute can throw exceptions which results with empty completion list * fix tests
* enable Roslyn's non blocking code complete update Microsoft.CodeAnalysis.xxx packages to version 2.0.0-rc3-61225-01 * roll back to Microsoft.CodyAnalysis.xxx RC2 nuget packages
* Update tutorial script to include immutability and pipelines * Updates/feedback
* Modify before_install.sh for OSX compatibility Modify before_install.sh so it works on OSX. The script now performs rudimentary OS detection, and Linux-specific commands are only executed when running on Linux. * Fix before_install for POSIX shells. Use the correct string comparison operator (= instead of ==). Modified the command substitution where certmgr is called so it uses the modern $(...) syntax instead of backticks.
* Minor spelling fixes and more print functions * Remove parenthetical as per @dsyme's feedback
Timing for 100k checks of the path "c:\dev\myproject\utilities\longstring with spaces\filename.fs": Original ~250ms This method ~100ms
* use Roslyn line numbers only in lexer cache * fix and refactor getSymbolAtPosition * fix after merge * fix out of range exn
This reverts commit c505000.
…ct(s) the symbol is declared (#2139)
* use asyncMaybe everywhere * fix compilation * continue the refactoring * finish the refactoring * fixed: AsyncMaybe.Delay was not lazy * fix tests
* Reduce number of calls to ComputeSourcesAndFlags, add wait indicator to some long running project tasks * Project close might throw an exception too, so bundle them both up and re-raise them as an aggregate exception * Added IVsSolutionBuildManager5 and IVsThreadedWaitDialogFactory to VS mocks to fix tests * Hide some types/modules that were accidentally added to the public API surface
* fix Language and Product names in F1 help context * make "FSharp" a common constant
…d setters (#2146) * remove duplicated navigation bar items for properties with getters and setters * fix glyphs for class let bindings * better navigation bar glyphs * fix accessibility for members and let bindings
) * use Layout as a source data of classification related tasks * fix indentation * parse xml doc by hand instead of relying on VS service * revert back old API * fix portable build * fix output in FSI * fix printing of operators * update Surface test * internalize new types and modules * tag module/namespace as keywords * fix tooltip formatting for types with hidden representation * fix separator placement
* add glyph to Quick Info tooltip * fix tests compilation
* revert "Use Roslyn line numbers only in lexer cache" #2090 * fix OutOfRangeException in getCachedSourceLineData * fixed: lexer cache does not work at all * fix finding start line for relexing in getCachedSourceLineData
* add glyph for declarations in Find All References result window * fixed: ColorizationService may return duplicated spans which caused FindReferencesService to crash * remove #nowarn "1182" * fixed: FindReferencesService does not work on external symbols
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.
No description provided.