HasChangesAsync should return false if the document content did not changed #62849

tmat · 2022-07-21T23:44:54Z

tmat · 2022-07-21T23:45:10Z

src/Features/Core/Portable/EditAndContinue/EditSession.cs

…hanged

tmat · 2022-07-22T18:36:21Z

@davidwengier Rebased to 17.3

tmat marked this pull request as ready for review July 21, 2022 23:44

tmat requested a review from a team as a code owner July 21, 2022 23:44

dotnet-issue-labeler bot added the Area-Interactive label Jul 21, 2022

tmat force-pushed the HasChangesFixes branch from 777318f to 855b2ca Compare July 21, 2022 23:45

davidwengier reviewed Jul 22, 2022

View reviewed changes

src/Features/Core/Portable/EditAndContinue/EditSession.cs Show resolved Hide resolved

davidwengier approved these changes Jul 22, 2022

View reviewed changes

tmat enabled auto-merge (squash) July 22, 2022 18:18

tmat changed the base branch from main to release/dev17.3 July 22, 2022 18:32

tmat requested review from 333fred, JoeRobich and a team as code owners July 22, 2022 18:32

tmat changed the base branch from release/dev17.3 to main July 22, 2022 18:32

tmat added 3 commits July 22, 2022 11:35

HasChangesAsync should return false if the document content did not c…

117b685

…hanged

Comment

134452f

Test source generated docs

1be1651

tmat force-pushed the HasChangesFixes branch from 282dbb5 to 1be1651 Compare July 22, 2022 18:35

tmat changed the base branch from main to release/dev17.3 July 22, 2022 18:36

tmat disabled auto-merge July 22, 2022 18:37

tmat added 2 commits July 22, 2022 12:24

Fix rebase

bc7e147

Include PopulateChangedAndAddedDocumentsAsync in test

7c41c3b

tmat enabled auto-merge (squash) July 22, 2022 20:35

tmat merged commit a8c241a into dotnet:release/dev17.3 Jul 25, 2022

allisonchou mentioned this pull request Jul 30, 2022

[Automated] PRs inserted in VS build main-32729.218 #63087

Closed

tmat added this to the 17.3 milestone Aug 8, 2022

Provide feedback