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

Add multilevel lookup breaking change #28965

Merged
merged 3 commits into from
Apr 18, 2022
Merged

Conversation

gewarren
Copy link
Contributor

@gewarren gewarren commented Apr 8, 2022

Fixes #28836.

Copy link
Member

@IEvangelist IEvangelist left a comment

Choose a reason for hiding this comment

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

Looks good to me, :shipit: when you're ready.

@gewarren gewarren requested a review from tdykstra as a code owner April 18, 2022 18:42
Copy link
Contributor

@tdykstra tdykstra left a comment

Choose a reason for hiding this comment

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

LGTM, but I noted one thing that looked potentially confusing

@gewarren gewarren merged commit fbc2ed7 into dotnet:main Apr 18, 2022
@gewarren gewarren deleted the multilevel branch April 18, 2022 23:42
IEvangelist pushed a commit to IEvangelist/docs that referenced this pull request Apr 21, 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.

[Breaking change]: Multi-level lookup behavior is being disabled
5 participants