Skip to content

Conversation

@BillWagner
Copy link
Member

Update the Tuples sample for the C# 7.3 tuple equality feature.

Samples for dotnet/docs#3965

New samples have been added for explaining tuple equality.

This applies to dotnet/docs#3965
Copy link

@rpetrusha rpetrusha left a comment

Choose a reason for hiding this comment

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

Left one comment, @BillWagner. Otherwise, this is ready to merge.

Person sample = new Person("Bill", "Wagner");

Console.WriteLine("Starting Equality Tests");
Console.WriteLine($"nill equals nil: {nil == nil}");

Choose a reason for hiding this comment

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

In this and the next two lines, nill should be nil

@BillWagner BillWagner merged commit 3cbd421 into dotnet:master May 15, 2018
@BillWagner BillWagner deleted the csharp73-tuple-equality branch May 15, 2018 16:12
karelz pushed a commit to karelz/samples that referenced this pull request Aug 31, 2018
* update tuples project for C# 7.3

New samples have been added for explaining tuple equality.

This applies to dotnet/docs#3965

* finish the equality tests samples

* GetHashCode should use tuples too.

* respond to feedback.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants