Skip to content
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

Merge master into features/VarianceSafetyForStaticMembers #46512

Merged

Conversation

AlekseyTs
Copy link
Contributor

No description provided.

CyrusNajmabadi and others added 30 commits July 22, 2020 11:43
Switch 'use auto prop' to use NRT and make us more null safe.
Enable CSharpInteractiveCommands.VerifyPreviousAndNextHistory
Enable BasicLineCommit.CommitOnSave
E.g.
`[installDir]\Common7\IDE\CommonExtensions\Microsoft\ManagedLanguages\VBCSharp\LanguageServices\InteractiveHost\Desktop\InteractiveHost64.exe`
Remove wait for out of process start from OnLoadedAsync as it has not…
* Move to the .NET 5 SDK

This is necessary for us to test features like covariant returns because
we need to be able to target `net5.0` in our unit test projects.

Issues encountered in this update:

- WinRT is no longer supported in .NET 5. As a result we no longer multi-target
.NET Core in our WinRT test assembly as well as having to disabling a
few WinRT tests in others.  dotnet/runtime#37672
- Change our Debugger Proxy tests to react to a runtime change
dotnet/runtime@c362923
- Work around .NET5 P6 bug dotnet/sdk#11862
Rename types to not collide with the much more relevant Workspace equivalents
* Use explicit namespace for proprietary resources

The Roslyn code base originated with all of our resource reference /
impl DLLs being checked into a single utility project. At the point we
moved to Git and went OSS these single utility project was factored into
two projects for a number of reasons.

1. Keeping DLLs we didn't have license for private
1. Letting us put large binaries into a NuPkg file vs. polluting Git
   history

When this split occured the namespaces for the utility projects were not
updated. That means many types in our public and private resource
projects have the same name. This makes it **really** difficult to
determine where types and resources are coming from.

This change adds an explicit namespace to the proprietary resource
assembly. This will help make it clear which resources come from it.

* Fix version

* Remove version ambigutiy

* Fix test failure
Replacing ILanguageServerProtocol with AbstractRequestHandlerProvider.
Refactoring common Extensions method code into generic helpers.
…eview1-to-release/dev16.8-preview1-vs-deps

Merge release/dev16.8-preview1 to release/dev16.8-preview1-vs-deps
…eview1-to-master

Merge release/dev16.8-preview1 to master
…/covariant-returns

Merge master to features/covariant-returns
…eview1-vs-deps-to-master-vs-deps

Merge release/dev16.8-preview1-vs-deps to master-vs-deps
…/covariant-returns

Merge master to features/covariant-returns
…/covariant-returns

Merge master to features/covariant-returns
…#45993)

* Support nullable annotations on unconstrained type parameters

Support `?` annotations on unconstrained type parameter references:
- `?` annotation is allowed for type parameters that are not constrained to reference types or value types
- `?` annotation is an error when used on unconstrained type parameter with `/langversion:8`

Support `where T : default` constraint:
- `default` constraint is allowed for method type parameters in overridden or explicitly implemented methods to disambiguate between overloads defined in the base or interface
- `default` constraint is an error with `/langversion:8`
- `default` constraint is an error when used outside of method overrides or explicit implementations
- `default` constraint is an error when the type parameter is constrained

Fixes dotnet#29146
Report teh project name that we were unable to recover a symbol from.
akhera99 and others added 20 commits July 30, 2020 16:05
Added search option for inline parameter name hints
Also fix unintentional priority change for StartOnIdle.
check if EditorAdaptersFactoryService gives us a null buffer
Rework compiler server abstraction
* Updated Error Message for CS1673

* TODOify

* Revert "Updated Error Message for CS1673"

This reverts commit 68fc0c8.

* PR Changes

* PR changes

* Update SolutionState.TrackedGeneratorDriver.cs
…/covariant-returns

Merge master to features/covariant-returns
Update IThreadingContext to support safe asynchronous shutdown
* Add ServiceLayer.Test that overrides any other layer or workspace kind.

Mark all services used for testing with ServiceLayer.Test.
Remove WorkspaceKind.Test and set the kind of TestWorkspace to `host`.

* Move MockWorkspaceEventListenerProvider to Fatures test composition

* Enable solution crawler in diagnostic tests
Remove unnecessary conditional compilation
@ghost
Copy link

ghost commented Aug 3, 2020

Hello @AlekseyTs!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 12 minutes. No worries though, I will be back when the time is right! 😉

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approval

@ghost ghost merged commit 791db0c into dotnet:features/VarianceSafetyForStaticMembers Aug 4, 2020
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.