-
Notifications
You must be signed in to change notification settings - Fork 4.2k
Merge master to master-vs-deps #36922
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
Conversation
Co-Authored-By: Jason Malinowski <jason@jason-m.com>
…tudioProject.cs Co-Authored-By: Marius Ungureanu <teromario@yahoo.com>
Wire GetSpeculativeTypeInfo with nullability information
This changes the names of the C# and VB editors so that they have a name that matches the other editors and usage elsewhere.
Enable tests for fixed scenarios on Mono.
Make metadata properties more efficient
Another thousand cleanup suggestions in IDE
* Don't update analyzed nullability for param conversions * Fix test types
Consistently name C#/VB editors
Relax the rule that tries to get rid of this./me. in Roslyn code.
|
@RikkiGibson There are file conflicts in the merge request. Please have a look. |
|
This Linux_test coreclr failure looks spurious to me:
FYI @dotnet/roslyn-infrastructure |
ghost
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Auto-approval
This is an automatically generated pull request from master into master-vs-deps.
git fetch --all git checkout merges/master-to-master-vs-deps git reset --hard upstream/master-vs-deps git merge upstream/master # Fix merge conflicts git commit git push upstream merges/master-to-master-vs-deps --forceOnce all conflicts are resolved and all the tests pass, you are free to merge the pull request.