-
Notifications
You must be signed in to change notification settings - Fork 145
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 dacfx to latest preview #845
Conversation
@@ -185,7 +191,7 @@ public class DeploymentOptions | |||
ObjectType.LinkedServerLogins, | |||
ObjectType.Endpoints, | |||
ObjectType.ErrorMessages, | |||
ObjectType.Filegroups, | |||
ObjectType.Files, | |||
ObjectType.Logins, |
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.
Should we leave this to be explicit?
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.
Both will be present in options allowing user to change... only that Files will be excluded and groups included by default (following SSDT defaults).
(Files should not be by default included because they in most cases fail the deployment - one will have to explicitly exclude the changes regarding db files for it to work. Earlier it used to work because Filegroup meant both - files and filegroups)
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.
I see. Thanks makes sense. Thanks!
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.
…ot there previously)
…ot there previously)
This brings in