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

Update plan.md to clarify many-to-many #2125

Merged
merged 1 commit into from
Mar 3, 2020
Merged

Conversation

vslee
Copy link
Contributor

@vslee vslee commented Feb 19, 2020

There are multiple GH issues tracking the various components of many-to-many. Previously, only one of them was mentioned here. I have added the rest. Please feel free to adjust/edit as you see fit. (I couldn't find an issue for the configuration sugar)
I have also added additional text to clarify which issues are planned and which are only stretch goals.

There are multiple GH issues tracking the various components of many-to-many. Previously, only one of them was mentioned here. I have added the rest. Please feel free to adjust/edit as you see fit. (I couldn't find an issue for the configuration sugar)
I have also added additional text to clarify which issues are planned and which are only stretch goals.
@ajcvickers
Copy link
Contributor

@vslee I like the idea here! Not merging immediately because I want to think about the exact language and formatting, since this is an important doc.

@vslee
Copy link
Contributor Author

vslee commented Feb 22, 2020

Yes of course. These changes make the style of that section different than the others, so they likely need to be revised to conform better.

@ajcvickers ajcvickers merged commit 4fcbfba into dotnet:master Mar 3, 2020
@ajcvickers
Copy link
Contributor

This now merged. Thanks!

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