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

Revert "Implement GetTypesByMetadataName" #58784

Merged
merged 1 commit into from
Jan 11, 2022

Conversation

333fred
Copy link
Member

@333fred 333fred commented Jan 11, 2022

Didn't make it in before the 17.1 bar closed. Reverting for 17.1 and we'll revert the revert in main.

@333fred 333fred requested review from a team as code owners January 11, 2022 22:24
@333fred 333fred enabled auto-merge (squash) January 11, 2022 22:24
@333fred
Copy link
Member Author

333fred commented Jan 11, 2022

@dotnet/roslyn-infrastructure for a review.

@jasonmalinowski
Copy link
Member

@333fred @RikkiGibson Was this the only thing merged that didn't make it in? In the past we've sometimes just done a force push of the release branch to bring it back in sync, which avoids reverts of reverts, but that has it's own fun.

@333fred
Copy link
Member Author

333fred commented Jan 11, 2022

@333fred @RikkiGibson Was this the only thing merged that didn't make it in? In the past we've sometimes just done a force push of the release branch to bring it back in sync, which avoids reverts of reverts, but that has it's own fun.

It is the only thing.

@333fred
Copy link
Member Author

333fred commented Jan 11, 2022

However, given that this has already gone into -vs-deps, I think the cleanest thing is to just merge the revert, then revert the revert in main.

@333fred 333fred merged commit 2eab822 into release/dev17.1 Jan 11, 2022
@333fred 333fred deleted the revert-58366-gettypesbymetadataname branch January 12, 2022 00:07
333fred added a commit to 333fred/roslyn that referenced this pull request Jan 12, 2022
…pped

* upstream/release/dev17.1:
  Revert "Implement GetTypesByMetadataName (dotnet#58366)" (dotnet#58784)
333fred added a commit that referenced this pull request Jan 12, 2022
333fred added a commit that referenced this pull request Jan 13, 2022
333fred added a commit that referenced this pull request Jan 13, 2022
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.

3 participants