-
Notifications
You must be signed in to change notification settings - Fork 551
Disable DependentUpon convention for EntityFramework.csproj #1312
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
Conversation
Code generated by Resources.tt->Resources.cs assumes a resource naming convention that isn't compatible with the DependentUpon convention, so disable it for this project.
|
Still seeing the test failures:
|
| <AssemblyVersion>6.0.0.0</AssemblyVersion> | ||
| <DefineConstants>$(DefineConstants);INTERNALS_INVISIBLE</DefineConstants> | ||
|
|
||
| <!-- Properties\Resources.tt expects a path that does not match the class in Properties\Resources.cs --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it worth changing the code to use the conventional name?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a great question I leave up to the maintainers. It didn't seem trivial, but I don't know the history of the Resources.tt approach or the details of its implementation.
8ca145a to
7b54e22
Compare
7b54e22 to
d41aa34
Compare
|
So it looks like I fixed almost all the errors, but there's two instances of this: @wtgodbe do you think that's resource-related? |
|
@ajcvickers, do you know (or know who might know) what might be up with that remaining error? Is it likely to be related to resource naming? |
|
@bricelam @rainersigwald @dougbu This was happening while I was on vacation; sorry for missing it. Does this change need to go in for the 3.1 release train? |
|
I believe @wtgodbe incorporated this proposal in #1291. @rainersigwald / @wtgodbe do we need this PR for something else? |
|
This PR has been closed because EF6 is no longer being actively developed. We are instead focusing on stability of the codebase, which means we will only make changes to address security issues. See the repo README for more information. |
Code generated by Resources.tt->Resources.cs assumes a resource naming convention that isn't compatible with the DependentUpon convention, so disable it for this project.