Skip to content

Merge master to features/param-nullchecking#36077

Merged
176 commits merged intofeatures/param-nullcheckingfrom
merges/master-to-features/param-nullchecking
Jun 4, 2019
Merged

Merge master to features/param-nullchecking#36077
176 commits merged intofeatures/param-nullcheckingfrom
merges/master-to-features/param-nullchecking

Conversation

@dotnet-bot
Copy link
Copy Markdown
Collaborator

This is an automatically generated pull request from master into features/param-nullchecking.

git fetch --all
git checkout merges/master-to-features/param-nullchecking
git reset --hard upstream/features/param-nullchecking
git merge upstream/master
# Fix merge conflicts
git commit
git push upstream merges/master-to-features/param-nullchecking --force

Once all conflicts are resolved and all the tests pass, you are free to merge the pull request.

mavasani and others added 21 commits May 31, 2019 07:43
Avoid renaming methods that don't change return type
Respect code style option severity in ConvertSwitchStatementToExpress…
Use diagnostichelper and properly pass along severity.
* Use IVsSolution to look up IVsHierarchy by project GUID

* Fix

* Fix mocks

* Add missing file

* Fix bad merge

* Fix merge
Fix import completion telemetry
Additionally, we do not mark the package reference to in Features.csproj as PrivateAsset as this leads to the assembly not getting included in the Features NuGet package and causes Omnisharp's analyzer execution to throw missing dependency assembly load errors.
Also update ExpectedDependency in DevDivInsertionFiles.csproj
Upgrade Microsoft.CodeAnalysis.FlowAnalysis.Utilities PackageReference to latest version
Add Default Interface Implementation tests for VB

Related to #35820, #35821, #35823, #35824, #35827, #35834, #35852, #35885, #35911, #35948.
Don't offer 'convert if to conditional' if it crosses a preprocessor directive.
…o merges/master-to-features/param-nullchecking
Copy link
Copy Markdown

@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 54b6387 into features/param-nullchecking Jun 4, 2019
@ghost ghost deleted the merges/master-to-features/param-nullchecking branch June 4, 2019 00:00
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.