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 to features/interpolated-string-constants #45655

Conversation

dotnet-bot
Copy link
Collaborator

This is an automatically generated pull request from master into features/interpolated-string-constants.

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

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

allisonchou and others added 19 commits July 1, 2020 12:09
…ecords with the latest design. (#45518)

Related to #45296.

From specification:
The record type includes a synthesized override of object.GetHashCode(). The method can be declared explicitly. It is an error if the explicit declaration is sealed unless the record type is sealed.
…624.3 (#45515)

Microsoft.Net.Compilers.Toolset
 From Version 3.7.0-4.20319.6 -> To Version 3.7.0-4.20324.3

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Fix tests broken by two PRs "crossing".
* Update AttributeTests.cs

* Update NullableReferenceTypesTests.cs

* Update NullableReferenceTypesTests.cs

* Fixed NullPointerException

* Removed duplicate test

* PR changes

* Added Robustness and Nullable Checks

* PR Changes

* Moved test to avoid merge conflict
Fixes #45465
Ensure we normalize the target scope passed to target symbol resolver. It doesn't handle the recently added 'namespaceanddescendants' scope in legacy target string format and expects it to be normalized to 'namespace' scope.
Fix issue preventing going to def on a complicated metadata symbol involving nullability and generics
Handle invalid dotnet_diagnostic entry without ID part in configurati…
…in records with the latest design. (#45601)

Related to #45296.

From specification:
If the record type is derived from a base record type Base, the record type includes a synthesized override of the strongly-typed Equals(Base other). The synthesized override is sealed. It is an error if the override is declared explicitly. The synthesized override returns Equals((object?)other).

This change also adjusts code-gen to properly compare EqualityContracts.
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 eaec4c6 into features/interpolated-string-constants Jul 5, 2020
@ghost ghost deleted the merges/master-to-features/interpolated-string-constants branch July 5, 2020 01:50
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.