-
Notifications
You must be signed in to change notification settings - Fork 228
Add compileBuiltIn property to project.json #1289
Conversation
@glennc @victorhurdugaci @lodejard @ChengTian |
string projectFilePath, | ||
string name, | ||
string legacyName, | ||
IEnumerable<string> fallbackIncludeing = null, |
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.
typo in fallbackIncludeing
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.
Will fix. Thanks.
This shouldn't break people that respecify the pattern today right? |
Respecify pattern won't break the application. Tested all projects in universe. And the internally the duplicate the patterns will be removed. But there is possibility that it will breaks in one situation that is when user explicitly overwrite But the chance is low. |
@davidfowl sign off? Ping |
@@ -88,6 +89,20 @@ public void FilesPatternsAreSet() | |||
Assert.Equal(new string[] { "buggy/*.*", "bin/**", "obj/**", "**/*.kproj", "no_pack/*.*" }, target.ContentPatternsGroup.ExcludePatterns); | |||
} | |||
|
|||
[Fact] | |||
public void RewriteCompileBuiltIn() |
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.
Add a test that does:
{
"compile": ["**/*.cs"]
}
Two tests added. |
Ping |
#1288