* add WriteCodeFragment build task for assembly-level attribute generation

* move InternalsVisibleTo to the project files where possible

This fixes build issues when older versions of MSBuild are used.

* Generate source for .resx files on build. (#3607)

* add build task to generate *.fs from *.resx files

* generate source for embedded resources in tests

* generate source for embedded resources in FSharp.Editor

* generate source for embedded resources in FSharp.LanguageService

* generate source for embedded resources in FSharp.ProjectSystem.FSharp

* generate source for embedded resources in FSharp.VS.FSI

* don't generate non-string resources when <=netstandard1.6

* update baseline error message for tests

The error output should be the exception message, not the exception type.

* perform up-to-date check before generating *.fs from *.resx

* remove non-idiomatic fold for an array comprehension

* correct newline replacement

* output more friendly error message

* throw if boolean value isn't explicitly `true` or `false`

* only generate object resource code on non `netstandard1.*` and `netcoreapp1.*` platforms

* ensure FSharp.Core specifies a target framework for resource generaton

* rename attributes to be non-ambiguous and properly include them

* fix order of file items in FSharp.Core

* Fix build.cmd for certain always-shown errors (like "unable to find registry key"), improve finding of VS2017 tools, fix DEVGUIDE.md (#3635)

* Fix build.com displaying the following error on each run "ERROR: The system was unable to find the specified registry key or value."

* Remove warning about reg.exe errors introduced in #3614 (in commit b548bd7, but unrelated to that fix), it is no longer necessary.

* Fix #3638, VS2017 Preview installation was not found when VS2017RTM is not installed. Remove comment introduced in #3614 (through commit 966bd7f7)

* Fixing JaroWinkler tests with InvariantCulture and fixing async tests by removing Debugger.Break() (#3627)

* Fixing JaroWinkler tests to use InvariantCulture for number-to-string

* Fixing the crashing of test runners because of a Debugger.Break() in a test

* update to System.Collections.Immutable 1.3.1 (#3641)

* update to System.Collections.Immutable 1.3.1

* fixes

* fix assembly reference

* [WIP] Adds optimized emit for int64 constants (#3620)

* Adds optimized emit for int64 constants.

* Adds comment linking to the changing PR.

Thanks for this PR.

Kevin

* fix assembly reference (#3646)

* Remove a few more build warnings (#3647)

* fix assembly reference

* remove more build warnings

* fix build

* move BuildFromSource projects to their own directory

* Adds tests for emitted IL for new Int64 constants. (#3650)

* Enable FS as prefix and ignore invalid values for warnings (#3631)

* enable fs as prefix and ignore invalid values for warnings + tests

* Allow #pragma to validate warnings

* do it right

* use ordinal compare

* In both places

* Add fs prefix to warnaserror

* Fixup tests

* Fix stack overflow on assembly resolution (#3658)

* Fix stack overflow on tp assembly resolution

* Feedback

* Add impl files to file check results (#3659)

* add LanguageServiceProfiling project to internals visible to list of FSharp.Compiler.Private project

* add ImplementationFiles to FSharpCheckFileResults

* make FSharpImplementationFileContents ctor internal

* throw if ImplementationFiles is called having keepAssemblyContents flag set to false

* add a test

* spelling and cosmetics

* This adds backup, restore, coloration and many more checks to the update-vsintegration.cmd (#3672)

* This adds backup, restore, coloration and many more checks to the update-vsintegration.cmd

* This adds backup, restore, coloration and many more checks to the update-vsintegration.cmd

* Remove ambiguous an irrelevant instruction, improved help and instructions

* Fix a scenario where the return code wasn't nonzero for error conditions, fixes not creating backup dir when not backing up

* add LanguageServiceProfiling project to internals visible to list of FSharp.Compiler.Private project (#3657)

* bump FCS version (#3676)

* bump version

* Update RELEASE_NOTES.md

* Parsing improvements: no reactor, add parsing options, error severity options (#3601)

* Parse without reactor, add parsing options, error severity options

* Revert parsing APIs (fallback to the new ones), fix VFT projects

* Cache parse results after type check

* Add impl files to file check results (#3659)

* add LanguageServiceProfiling project to internals visible to list of FSharp.Compiler.Private project

* add ImplementationFiles to FSharpCheckFileResults

* make FSharpImplementationFileContents ctor internal

* throw if ImplementationFiles is called having keepAssemblyContents flag set to false

* add a test

* spelling and cosmetics

* This adds backup, restore, coloration and many more checks to the update-vsintegration.cmd (#3672)

* This adds backup, restore, coloration and many more checks to the update-vsintegration.cmd

* This adds backup, restore, coloration and many more checks to the update-vsintegration.cmd

* Remove ambiguous an irrelevant instruction, improved help and instructions

* Fix a scenario where the return code wasn't nonzero for error conditions, fixes not creating backup dir when not backing up

* add LanguageServiceProfiling project to internals visible to list of FSharp.Compiler.Private project (#3657)

* bump FCS version (#3676)

* bump version

* Update RELEASE_NOTES.md

* updates to make tests pass

* restore old behaviour of CheckFileInProjectAllowingStaleCachedResults (builder had been created by ParseFileInProject)

* restore use of CheckFileInProjectAllowingStaleCachedResults

* deprecate test relying on whacky behaviour of deprecated GetCheckResultsBeforeFileInProjectEvenIfStale

* Use ParseFile and FSharpParsingOptions instead of ParseFileInProject

* prepare FCS release with this feature

* whitespace cleanup (#3682)

* whitespace and docs (#3684)

* Preserve XML docs for in-memory project references (#3683)

* fix xmldocs for in-memory project references

* add test

* fix tests

* whitespace and comments (#3686)

* fix assembly reference

* whitespace and comments

* whitespace and comments

* whitespace and comments

* cherry pick two PRs from FCS (#3687)

* fix assembly reference

* remove line endings from all *.nuspec files

* ProjectCracker returns *.fsi files in FSharpProjectOptions.SourceFiles array (in addition to *.fs files, in right order)

* ProjectCracker raises exception if ProjectCrackerTool returns non null ProjectCrackerOptions.Error (new field)

* fix build on linux

* fix a test

* slashes

* revert slashes

* Update FSharp.Compiler.Service.ProjectCracker.nuspec

* try to fix travis

* try to fix travis

* list dependencies

* no obsolete pdb in nuget

* list dependencies

* cherry pick of fsharp/fsharp change

* bump FCS version number (#3688)

* Update FSharp.Compiler.Service.MSBuild.v12.nuspec

* fix FCS nuget on windows

* fix-resource (#3690)

* Bump FSharp.Compiler.Tools to 4.1.27 and align mono build files (#3693)

* ri change from fsharp

* fix test

* bump FSC tools to 4.1.27

* remove fsharp.core from Mono GAC

* align mono directory

* fix typo

* install back versions with Mono

* fix typo

* update FCS doc generation (#3694)

* update DEVGUIDE to add addiitional steps before running build (#3725)

* Split templates out into a seperate vsix (#3720)

* merge error

* Merge issues

* update to System.Collections.Immutable 1.3.1

* fixes

* add build task to generate *.fs from *.resx files

* generate source for embedded resources in tests

* generate source for embedded resources in FSharp.Editor

* generate source for embedded resources in FSharp.LanguageService

* generate source for embedded resources in FSharp.ProjectSystem.FSharp

* generate source for embedded resources in FSharp.VS.FSI

* don't generate non-string resources when <=netstandard1.6

* update baseline error message for tests

The error output should be the exception message, not the exception type.

* perform up-to-date check before generating *.fs from *.resx

* remove non-idiomatic fold for an array comprehension

* correct newline replacement

* output more friendly error message

* throw if boolean value isn't explicitly `true` or `false`

* only generate object resource code on non `netstandard1.*` and `netcoreapp1.*` platforms

* ensure FSharp.Core specifies a target framework for resource generaton

* rename attributes to be non-ambiguous and properly include them

* Add editor formatting service for auto-deindent

* Minor refactor of the indentation service - do not indent after 'function'

* Only use smart indentation if indent style is set to 'Smart'

* Fix broken unit test build

* Implement review comments, fix build

* Fix some broken brace matching tests

Still WIP, other tests still broken

* Fix failing indentation tests

* Add formatting service tests

* Add more brace matching tests

Fixes #2092

Issue opened:  https://github.com/Microsoft/visualfsharp/issues/3539

This reverts commit 0c95431c.

This reverts commit fe014083, reversing
changes made to cf4cd7f6.

This was removed from recent Roslyn builds and since very little functionality was used, the appropriate methods were simply ported over.

* import remaining FCS code

* add missing files

* fix some parts of build

* more FCS integration work

* build fcs.dll and vs targets

* revert changes to proj files

* update sig files

* various cleanup items

* simplify #ifs

* fix various tests

* enable scripting testing

* fix build

* fix build

* fix tests and build

* fix tests and build

* fix build

* fix tests

* fix tests

* add diagnostic and fix coreclr build

* fix printing tests

* test updates

* improve error logging

* integrate visualfsharp master

* dependency files

* proj file

* dependency files

* fix build

* fix fsi.exe in GUI mode

* add end of line to source text if needed as before

* fix many tests

* fix remaining tests

* better strong name logic to fix final test

* reinstate all strong names

* fix last tests

* fix last test again

* ReplaceUnusedValueWithUnderscore code fix replaces "this" var with double underscore instead of single one

* do not show code fix for anything but funcOrMemberOrValue

do not suggest to rename members with underscore
show multiple suggestions for all symbols on same line

* Show symbol name in RenameUnusedValue code fix action name

* No longer use Old Lang Service XMLDocumentation in FSharp.Editor and its tests

* Remove needless stuff and add resource

* Use SR properly

* Structure.fs cleanup

* move block structure service logic to FSharp.LanguageService.Compiler

* add UnusedDeclarationsAnalyzer

* fix compilation

* make UnusedDeclarationsAnalyzer work on private symbols only

* fix compilation

UnusedDeclarationsAnalyzer analyze semantic, not syntax

* finish UnusedDeclarationsAnalyzer

* do not mark as unused symbols prefixed with underscore

* fix FSharp.Editor.Pervasive.isScript

* UnusedDeclarationsAnalyzer consider all declaration in scripts as private to file

* Added "You must add a reference to 'foo'" code fixer

* Preliminary work to enable metadata reference code fixing

* Fix SetupProjectFile passing in the wrong IVsHierarchy to CreateProjectContext

* Localise code fix strings, add support for metadata (assembly) references

* Fix removing references not being reflected in the Roslyn workspace

* Added "You must add a reference to 'foo'" code fixer

* Preliminary work to enable metadata reference code fixing

* Rebased onto master

* Localise code fix strings, add support for metadata (assembly) references

* Fix removing references not being reflected in the Roslyn workspace

* Fix dodgy merge

* Fix reference to CommonRoslynHelpers

* Removed intellisense strings accidentally added in merge

* Ignore case when comparing assembly names

* Fix dodgy merge

* settings store and editor options

* show CLIMutable record use, guid constants

* saner composition

* fix CompletionProvider tests

* fix public surface area

* localization

* move xaml into separate project

* updated vsintegration readme

* correct type in change handler

* use Newtonsoft's PopulateObject for cheap schema evolution

* cleanup added project

* Revert "cleanup added project"

This reverts commit 163ebdaabc0d3b206fc436b0b47a622f91e20658.

* cleanup, another attempt

* split out language service into dedicated dir

* dedicated extensions module

* trim pervasive

* rename constants

* symbols module

* typedastutils module

* FSharpChecker extensions

* rename roslyn helpers

* editor rename cleanup

* fix test renaming

* full type name in tooltip, provisional tab preferred

* more entities made navigable

* use IGoToDefinition service

* this is used only here

* MEF import FSharpGotoDefinitionService into QuickInfoProvider

* speed up gotoDefinition

* additional GotoDefn navigation strategies

* quickinfo navigation stays in its lane

tooltip from .fsi links to .fsi
tooltip from .fs links to .fs
quick navigation if no redirect is necessary

* fix unittests

* restore recursive matchingDoc

* asynchronous navigation from tooltips

* fix cross project .fs -> .fs and .fsi -> .fsi Navigation

* cleanup and extra documentation

fixed bug in cross project .fs -> .fs navigation

* missed this one

* gotodefinition sig <-> impl at declaration location

* fix async workflow

* animate status bar search and timeout on msgs

* Better links styling

* integrate sig doccoms

* fix error introduced by prior merge

* fixed invalid type access in `getUnusedOpens`

* fix invalid span bug in `symbolIsFullyQualified`

* check if normalized doccom text matches

* cleanup status bar usage

* fix underline pen position, code cleanup and formatting

* do not show links for symbol itself

* Enable sourcelink for repo

* Make it turn offable

* Somehow I deleted microbuild=true

* remove specific symbol type settings from a couple of projects

* Enable sourcelink for build

* Immutable fix

* More tweaks

can build and install new VSIX from the commandline without having the
VSSDK installed

* wip

* port Roslyn's C# completion triggering logic as is

* fixed: completion do not trigger at all

* fix position

* fix a test

* do not provide completion on certain places

* fix muting completion on Named(Wild) pat in binding

* do not provide completion on ctor / lambda parameter names + tests

* fix binding traversing in completion

* fix tests

* fix the last failing test

* wip

* wip

* fix compilation

* the list kind of works

* try to add open (fake)

* insert fake open statement

* works for dummy ns insert position

* it works (draft)

* wip

* pass proper namespace to open

* do not suggest operators

prefer already resolvable items

* fix compilation

* fix

* filter out unqualified types if there are same normal ones (UnqualifiedType (tcRef1 :: _) is equal to TType.Type_app (_, tcRef1) is tcRef1 = tcRef2)

* fix compilation

* do not open Microsoft.FSharp.xxx namespaces

add mandatory qualifier, which makes completion works for symbols like `Printf.kprintf`

* fix bugs

* wip

* wip

* Revert "wip"

This reverts commit fe3247e08fccabc00242f658282565f568eb2f3b.

* fixing bugs with RCA modules

* bug fixing

* fix filter text

* optimize completion list sorting

* code cleanup and doc

* defer getting all entities (do not get them after dot completion at all)

* fix compilation

* remove dead code

* rename RawEntity to AssemblySymbol and use FSharpSymbol instead of Item in it

* do not call getAllSymbols unless necessary (now for real)

* AssymblySymbol -> AssemblySymbol

* add completion for keywords

* order completion items by IsResolved property

* add CompletionContext.OpenDeclaration, remove IsAtOpenDeclaration ad-hoc filtering

do not show keywords at open declaration position

* parameterize config with envars, vssdk nuget

- 'RoslynVSBinariesVersion' will also set the nuget version number
- add nuget packages for VSSDK references to config

* change vssdk refs in projects to nuget packages

* envars in config files prevent proper restore, back to raw strings

* fix projectaggregator paths

* fix package compatibility issues with extern alias

* added missing references

* fix VisualStudio.Shell version conflicts with extern alias

* only use VSShell15 for ProjectSystem.Base

* build successful using all nuget VSSDK packages

* update proto package refs

* fix typo

* align package versions

* remove aliases from LanguageService.Base

unnecessary after removing reference to VisualStudio.Shell.14.0

* removed unneeded references, aligned more refs

* align netcore packages, fix json formatting

* add pkg ref for Compiler.Server.Shared build

* update Microsoft.Build packages

* use VSSDK buildtools package for F# VSIX

* custom tooltip test

* tagClass augmented

* kinda works

* getting there

* module, alias

* not sure if works

* fix build

* go away!

* parens out

* halleluyah it builds

* blank space on empty doc removed

* more sensible

* handle linked files

* collapse empty TextBlocks

* styling, simple tooltip, handle scripts

* dismiss quickinfo on navigation

* reliably dismiss quickinfo session

* interface instead of boxing

* port Roslyn's C# completion triggering logic as is

* fixed: completion do not trigger at all

* fix position

* fix a test

* do not provide completion on certain places

* fix muting completion on Named(Wild) pat in binding

* do not provide completion on ctor / lambda parameter names + tests

* fix binding traversing in completion

* fix tests

* fix the last failing test

* "trigger completion after a character is typed" setting in a new completion property page

* force intellisense property page loading

* do not commit completion on space in auto triggering mode

* fix completion list ordering

* show proper glyphs for namespaces and structures

* glyphs for literals and delegates

* fix layout for struct records and unions

* proto does not like Option.defaultValue

* show [<Struct>] in layout instead of "struct" for struct records and DUs

* show proper glyph for operators in QuickInfo

* fix test compilation

* layout `Measure` and `Struct` special attributes as types

prettier records layout

* fix a test

* Revert records layout change

* fix TypeChecker

* do not print [<Struct>] for ordinary structures

* wip

* it kind of works

* bug fix

* remove unused code

* trigger completion on /

* everything works

* cosmetics