* No errors for non virtual members overrides

* Check for isFSharpObjModelTy as we are interested on csharp virtual methods

* More testing

* Add a feature lang preview

* More testing

* Fix PR comment

* Small deperecations

* fantomas

* Try fix Symbol.PrivateToFile

* fantomas

* More tests

Fixes https://github.com/dotnet/fsharp/issues/14275

* Symbols: add BasicQualifiedName, FSharpType.ErasedType
Co-authored-by: NTomas Grosup <tomasgrosup@microsoft.com>

* Add attribute to prevent fsc inlining but allow JIT inlining
* Work around IL checker shortcomings
* Update surface area
* Rename NoFscInliningAttribute

* Pop string nesting before error in nested quotes

* Cover more cases of incomplete interpolation

* Add tests for unmatched curly brace

Two tests for unmatched curly brace in interpolated string

* Split tests into separate cases

This splits two unit tests that were testing multiple cases in one go
into multiple unit tests - one per each case.

Note, that original tests were expecting an accurate error for each
of multiple lines of malformed string literals. They would no longer
pass with changes introduced by previous commits in this PR.
Co-authored-by: NAdam Boniecki <adboniec@microsoft.com>

* Add GetSetKeyword to trivia

* Update failing unit test to only include the `set` range.

* GetSetKeywords

* Update base line for test

Co-authored-by: NTomas Grosup <tomasgrosup@microsoft.com>

* Allow extension methods without type attribute.
* Add Extension attribute to type if any member has the attribute.
* Handle C# extensions as let bindings in a module.
* Add test for recursive types.
* Add tests for types in recursive modules.
* Only add attribute to module when module declaration has attribute.
* Add additional test.
* Add CSharp Extension Attribute Not Required as a Preview Language Feature
* Passing propper Fsharp.Core version to executed tests in legacy FsharpSuite
* Fix for tests that use NetFramework types (like winforms brushes etc.)
Co-authored-by: NEdgar Gonzalez <edgar.gonzalez@fundourselves.com>
Co-authored-by: NTomas Grosup <tomasgrosup@microsoft.com>
Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>

* Use more accurate range to only report  obsolete field as a obsolete rather that making the whole record construction expression obsolete

* Use record expression range for CheckRecdFieldAccessible

Co-authored-by: NDon Syme <dsyme@users.noreply.github.com>

* fix metadata failure due to double duplication

* fix metadata failure due to double duplication

* Filter completions on abstract member type signature

Co-authored-by: NDon Syme <dsyme@users.noreply.github.com>
Co-authored-by: NTomas Grosup <tomasgrosup@microsoft.com>
Co-authored-by: NDon Syme <donsyme@fastmail.fm>
Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>

* Refactor TypeKeyword to LeadingKeyword in SynTypeDefnTrivia.
* Refactor SynModuleOrNamespaceTrivia to use LeadingKeyword type.

* Improve Unclear error FS3204 multicase union case struct

* Fix 14097

* Fix 14097

Co-authored-by: NPetr Pokorny <petrpokorny@microsoft.com>
Co-authored-by: NTomas Grosup <tomasgrosup@microsoft.com>
Co-authored-by: NDon Syme <dsyme@users.noreply.github.com>

* Changing warning to implementing interfaces within intrisincs
* In case of recursive namespaces and modules, warning is not generated
* Adding tests for combinations of rec and non-rec modules/namespaces/nested namespaces

* Add failing test

* Attempt to print attributes after and keyword.

* Update indentation in unit test.
Co-authored-by: NTomas Grosup <tomasgrosup@microsoft.com>

* fix unqualified name resolution for nested types in presence of instance extension members

* add tests

* add tests

* apply formatting
Co-authored-by: NTomas Grosup <tomasgrosup@microsoft.com>
Co-authored-by: NKevin Ransom (msft) <codecutter@hotmail.com>

* Use MSBuild's logging facilities instead of writing to console.

* Changing usage of WriteCodeFragment in tests

* Applying cleaner formatting
Co-authored-by: NTomas Grosup <tomasgrosup@microsoft.com>

* Add SynLeadingKeyword

* Add StaticLet and StaticLetRec.

* Add Do and StaticDo.

* Trigger CI

* Format SyntaxTrivia.

* Update Surface area.

* Trigger CI
Co-authored-by: NTomas Grosup <tomasgrosup@microsoft.com>

Compiler error for signature/implementation files when declaring multiple parameters with the same name in abstract methods (#14084)

Co-authored-by: NKevin Ransom (msft) <codecutter@hotmail.com>

Co-authored-by: NVlad Zarytovskii <vzaritovsky@hotmail.com>
Co-authored-by: NDon Syme <dsyme@users.noreply.github.com>

* Add SynType.Or for generic constrains in the form (^A or ^B):...

Co-authored-by: NTomas Grosup <tomasgrosup@microsoft.com>