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

wrong name of folder, issue #10006 #10007

Closed
wants to merge 2 commits into from
Closed

Conversation

Ligren
Copy link

@Ligren Ligren commented Dec 14, 2018

Should be Data, not Models.

Should be Data, not Models.
@scottaddie scottaddie self-requested a review December 14, 2018 15:28
@scottaddie
Copy link
Member

@Rick-Anderson I'm able to reproduce the problem in VS. This change looks good from that perspective. I'll let you be the final approver on this PR.

@Rick-Anderson
Copy link
Contributor

It's not that simple. The scaffolder puts it in the data folder but uses the Model namespace. The VSC and VS for mac put it in the models folder. See dotnet/Scaffolding#822

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.

None yet

3 participants