Skip to content

Remove obsolete UnitTesting external access APIs#45553

Merged
tmat merged 1 commit intodotnet:masterfrom
tmat:RemoveObsoleteUnitTestingApis
Jul 2, 2020
Merged

Remove obsolete UnitTesting external access APIs#45553
tmat merged 1 commit intodotnet:masterfrom
tmat:RemoveObsoleteUnitTestingApis

Conversation

@tmat
Copy link
Member

@tmat tmat commented Jun 30, 2020

No description provided.

@tmat tmat requested a review from a team as a code owner June 30, 2020 02:28
@tmat
Copy link
Member Author

tmat commented Jun 30, 2020

@shyamnamboodiripad @Cosifne PTAL

@tmat tmat added the Area-IDE label Jun 30, 2020
@Cosifne
Copy link
Member

Cosifne commented Jun 30, 2020

LGTM but I found one missing obsolete API in test impact.
I have a PR to remove it. So it would better to merge in this PR after this https://github.com/dotnet/testimpact/pull/2942

@tmat
Copy link
Member Author

tmat commented Jun 30, 2020

@Cosifne Can you ping me here when that PR is merged and inserted to VS?

@tmat tmat merged commit ec72c14 into dotnet:master Jul 2, 2020
@ghost ghost added this to the Next milestone Jul 2, 2020
@tmat tmat deleted the RemoveObsoleteUnitTestingApis branch July 2, 2020 18:07
@JoeRobich JoeRobich modified the milestones: Next, 16.8.P1 Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants